Skip to content

[TT-14794] change behavior explanation when path does not match for a stream #6524

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 49 commits into
base: master
Choose a base branch
from

Conversation

pvormste
Copy link
Contributor

@pvormste pvormste commented Jun 11, 2025

User description

This PR changes the explanation of what happens when a path does not match on a streams API. This change is introduced for version 5.9.0 and 5.8.3.

Preview Link

https://deploy-preview-6524--tyk-docs.netlify.app/docs/nightly

Checklist

  • Added a preview link
  • Reviewed AI PR Agent suggestions
  • For Tyk Members - Added Jira DX PR ticket to the subject
  • For Tyk Members - Added the appropriate release labels (for fixes add the latest release)

New Contributors


PR Type

Documentation


Description

  • Updated explanation of stream path mismatch behavior.

  • Clarified that 404 is returned if no stream path matches.

  • Specified change applies from version 5.9.0 onwards.


Changes walkthrough 📝

Relevant files
Documentation
event-driven-apis.md
Clarify 404 response for unmatched stream paths in docs   

tyk-docs/content/api-management/event-driven-apis.md

  • Revised description for unmatched stream paths to state 404 response.
  • Removed outdated note about previous behavior.
  • Clarified middleware behavior for stream route matching.
  • +1/-1     

    Need help?
  • Type /help how to ... in the comments thread for any questions about PR-Agent usage.
  • Check out the documentation for more information.
  • Copy link
    Contributor

    ⚠️ Deploy preview for PR #6524 did not become live after 3 attempts.
    Please check Netlify or try manually: Preview URL

    Copy link
    Contributor

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    ⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ Recommended focus areas for review

    Documentation Accuracy

    Ensure the updated explanation about the 404 response for unmatched stream paths is correct for the specified versions and does not mislead users about behavior in earlier versions.

    *   **If No Match**: The `Stream Middleware` will respond with a `404 Not Found` status code.

    Copy link
    Contributor

    PR Code Suggestions ✨

    No code suggestions found for the PR.

    Copy link

    netlify bot commented Jun 11, 2025

    PS. Add to the end of url /docs/nightly

    Name Link
    🔨 Latest commit d2da60e
    🔍 Latest deploy log https://app.netlify.com/projects/tyk-docs/deploys/68754692890d900008abe15b
    😎 Deploy Preview https://deploy-preview-6524--tyk-docs.netlify.app
    📱 Preview on mobile
    Toggle QR Code...

    QR Code

    Use your smartphone camera to open QR code link.

    To edit notification comments on pull requests, go to your Netlify project configuration.

    @pvormste pvormste requested review from sharadregoti and removed request for sharadregoti June 12, 2025 08:27
    buger added 29 commits June 24, 2025 16:26
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Projects
    None yet
    Development

    Successfully merging this pull request may close these issues.

    3 participants