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

Go backend #277

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

Go backend #277

wants to merge 20 commits into from

Conversation

deddy2101
Copy link

as said this branch implements cloudflare access (or jwt signed with rs256) token
the code I added is in the jwt_gobck.go file, the code is provided with the tests i made. I am currently usin this plugin and i i find some bug i'll correct and remove them :)

@deddy2101 deddy2101 requested a review from iegomez as a code owner May 9, 2023 16:57
@iegomez
Copy link
Owner

iegomez commented May 20, 2023

Hey, @deddy2101, sorry for not checking this yet.
I'll try to read a bit about Cloudflare's JWT implementation and then take a look at the PR during the weekend.

Cheers!

@iegomez
Copy link
Owner

iegomez commented May 29, 2023

I did some work last week to enable auto testing on PRs, but it seems it was not retroactive.
Luckily, I'm not zealously in the "gradual commits" camp when it's small enough to review, so if you could locally squash them and force push everything in a single commit, I think that'll trigger checks again and updated tests will run. Can I bother you with that?

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.

3 participants