Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Update authenticated example - set token in echo context..#917

Merged
2 commits merged into
oapi-codegen:masteroapi-codegen/oapi-codegen:masterfrom
sblackstone:mastersblackstone/oapi-codegen:masterCopy head branch name to clipboard
May 5, 2023
Merged

Update authenticated example - set token in echo context..#917
2 commits merged into
oapi-codegen:masteroapi-codegen/oapi-codegen:masterfrom
sblackstone:mastersblackstone/oapi-codegen:masterCopy head branch name to clipboard

Conversation

@sblackstone

Copy link
Copy Markdown
Contributor

It was completely unclear to me how to propagate the parsed token back to the echo context so that it can be used in the handlers.

This small PR updates the example so that the property is set...

Comment thread examples/authenticated-api/echo/server/jwt_authenticator.go Outdated
@sblackstone

Copy link
Copy Markdown
Contributor Author

Can this be merged?

@ghost
ghost merged commit 8b1c1e2 into oapi-codegen:master May 5, 2023
adrianpk pushed a commit to foorester/oapi-codegen that referenced this pull request Jan 16, 2024
…gen#917)

* Update authenticated example so that we demonstate how to make the token available to the handlers

* Favor key instead of string
adrianpk added a commit to foorester/oapi-codegen that referenced this pull request May 31, 2024
…gen#917)

* Update authenticated example so that we demonstate how to make the token available to the handlers

* Favor key instead of string
danicc097 pushed a commit to danicc097/oapi-codegen that referenced this pull request Aug 31, 2024
…gen#917)

* Update authenticated example so that we demonstate how to make the token available to the handlers

* Favor key instead of string
This pull request was closed.
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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.