Skip to content

Conversation

david-mohr
Copy link
Contributor

Adds an optional mechanism parameter and implements EXTERNAL auth following the Java reference implementation.

Also fixes a broken test.

@icappello
Copy link
Member

Hello, thanks for opening this PR! We will give you a feedback in the next few days

@david-mohr
Copy link
Contributor Author

Just about to head on vacation, so I will be able to make these updates in a few weeks.

@david-mohr
Copy link
Contributor Author

Is there a way I can run these github actions on my own account? I have not used github actions before, so not familiar with them. Plus I seem to be in a different timezone making it challenging to work on it while you are available.

@icappello
Copy link
Member

Hello David, I think you need to enable Actions in your forked repository by visiting the "Actions" tab. You should see a warning with a button to enable workflows.

Here is a discussion regarding this https://github.com/orgs/community/discussions/50736

Thanks again!

@david-mohr
Copy link
Contributor Author

Thanks for the pointers, the workflow should now be working as expected.

@icappello icappello requested review from l4mby and tarzacodes October 31, 2024 10:06
@icappello
Copy link
Member

We'll give you a feedback as soon as possible.

Thanks!

@gpad
Copy link
Contributor

gpad commented Nov 6, 2024

@david-mohr Thank for the PR I have added two small comments. WDYT about it?

@david-mohr
Copy link
Contributor Author

@gpad thanks, updates applied

@icappello icappello merged commit d8083e5 into coders51:main Nov 7, 2024
2 checks passed
@david-mohr
Copy link
Contributor Author

@icappello just curious, do you have an eta on the next release? Happy using my fork for now, but wanted to follow up before I forget. Thanks!

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