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

Merged
merged 54 commits into from
Jul 15, 2025

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 26a9ece
    🔍 Latest deploy log https://app.netlify.com/projects/tyk-docs/deploys/68765da222c20e000834a1f0
    😎 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
    @sharadregoti sharadregoti merged commit 2aa801c into master Jul 15, 2025
    8 checks passed
    @sharadregoti sharadregoti deleted the TT-14794-wrong-path-on-streams-api branch July 15, 2025 13:54
    @sharadregoti
    Copy link
    Contributor

    /release to release-5.8

    Copy link
    Contributor

    tykbot bot commented Jul 15, 2025

    Working on it! Note that it can take a few minutes.

    Copy link
    Contributor

    tykbot bot commented Jul 15, 2025

    @sharadregoti Created merge PRs

    buger added a commit that referenced this pull request Jul 15, 2025
    …ath does not match for a stream (#6524)
    
    [TT-14794] change behavior explanation when path does not match for a stream (#6524)
    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