Skip to content
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

Add upstream patches for rousette #866

Merged
merged 4 commits into from
Dec 11, 2024
Merged

Add upstream patches for rousette #866

merged 4 commits into from
Dec 11, 2024

Conversation

mattiaswal
Copy link
Contributor

This fix the issue when query an instance which have an identy-ref as a key (ietf-routing:routing/control-plane-protocols/control-plane-protcol)

Adding nginx_lua_support to be able to keep compatibility with python requests. It transform the urls, and remove all %3A, replace them with :

This fix #781

Description

Checklist

Tick relevant boxes, this PR is-a or has-a:

  • Bugfix
    • Regression tests
    • ChangeLog updates (for next release)
  • Feature
    • YANG model change => revision updated?
    • Regression tests added?
    • ChangeLog updates (for next release)
    • Documentation added?
  • Test changes
    • Checked in changed Readme.adoc (make test-spec)
    • Added new test to group Readme.adoc and yaml file
  • Code style update (formatting, renaming)
  • Refactoring (please detail in commit messages)
  • Build related changes
  • Documentation content changes
    • ChangeLog updated (for major changes)
  • Other (please describe):

Copy link
Contributor

@troglobit troglobit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work on this, also I'm so happy we found the root cause in Infamy!

@mattiaswal mattiaswal merged commit 76ab0fe into main Dec 11, 2024
6 checks passed
@mattiaswal mattiaswal deleted the update-rousette branch December 11, 2024 12:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Backport upstream Rousette identityref patches
2 participants