Skip to content

Conversation

@jack-r-hong
Copy link
Contributor

Hello, this is my first time contributing code. If you have any suggestions, please feel free to let me know. For OIDC Line Login, you only need to add id_token_key_type=JWK in the exchange step to issue tokens in HS256 format.

Related issue(s)

#1116

Checklist

  • I have read the contributing guidelines.
  • I have referenced an issue containing the design document if my change
    introduces a new feature.
  • I am following the
    contributing code guidelines.
  • I have read the security policy.
  • I confirm that this pull request does not address a security
    vulnerability. If this pull request addresses a security vulnerability, I
    confirm that I got the approval (please contact
    [email protected]) from the maintainers to push
    the changes.
  • I have added tests that prove my fix is effective or that my feature
    works.
  • I have added or changed the documentation.

Further Comments

@CLAassistant
Copy link

CLAassistant commented Dec 6, 2024

CLA assistant check
All committers have signed the CLA.

Copy link
Member

@aeneasr aeneasr left a comment

Choose a reason for hiding this comment

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

Thank you very much! Could you please also add to the docs a guide for Line? See here for an example: ory/docs#1693

Sorry for the long wait on your PR and congrats to your first submission! :)

@aeneasr aeneasr requested a review from a team as a code owner January 2, 2025 09:34
@aeneasr aeneasr enabled auto-merge (squash) January 2, 2025 09:35
@aeneasr aeneasr disabled auto-merge January 2, 2025 09:35
David-Wobrock added a commit to David-Wobrock/docs that referenced this pull request Jan 14, 2025
@David-Wobrock
Copy link
Contributor

Hello!

Thank you very much for this @jack-r-hong ❤️
I tested it locally, because we are looking to set up LINE, and I can confirm that it fixes the bug discussed in #1116 ✔️

In order to contribute, here is the documentation PR ory/docs#2006 @aeneasr
Hoping to see this being released soon 🤞 🙏

David-Wobrock added a commit to David-Wobrock/docs that referenced this pull request Jan 15, 2025
David-Wobrock added a commit to David-Wobrock/docs that referenced this pull request Jan 20, 2025
David-Wobrock added a commit to David-Wobrock/docs that referenced this pull request Jan 22, 2025
David-Wobrock added a commit to David-Wobrock/docs that referenced this pull request Jan 28, 2025
David-Wobrock added a commit to David-Wobrock/docs that referenced this pull request Feb 7, 2025
@alnr alnr merged commit 729effd into ory:master May 14, 2025
26 checks passed
@alnr
Copy link
Contributor

alnr commented May 14, 2025

Thank you!

David-Wobrock added a commit to David-Wobrock/docs that referenced this pull request May 15, 2025
alnr pushed a commit to ory/docs that referenced this pull request May 21, 2025
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.

5 participants