Skip to content

Conversation

@Vaibhav2154
Copy link

Description

This PR Fixes #

Notes for Reviewers

Signed commits
Yes, I signed my commits.

Signed-off-by: Vaibhav M N <vaibhavvaibhu2005@gmail.com>
Signed-off-by: Vaibhav M N <vaibhavvaibhu2005@gmail.com>
Signed-off-by: Vaibhav M N <vaibhavvaibhu2005@gmail.com>
@Vaibhav2154
Copy link
Author

@leecalcote Please review my pull request and let me know if there are any additonal changes to be made.

- Provide the desired configuration JSON through Meshery's UI or API.
- Deploy the filter and observe how it modifies JWT tokens based on your specifications.

For more details about configuring WASM filters with Meshery, visit the official ![Meshery documentation](https://github.com/meshery/.github/blob/master/profile/README.md)
Copy link
Member

Choose a reason for hiding this comment

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

Thanks, @Vaibhav2154.

The exclamation mark seems out of place here...

Copy link
Author

Choose a reason for hiding this comment

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

Screenshot_20250102_050537
@leecalcote the redirecting link to Meshery Documentation is working fine. So I guess that '!' is in place correctly itself.
If there's anything to be resolved further for getting the PR merged please let me know.

Copy link
Member

Choose a reason for hiding this comment

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

Think harder on this, please.

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.

2 participants