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

[type:refactor] plugin-jwt SPI extension for parsing JWT payloads to ensure compatibility with older versions of JWT tools #5882

Merged
merged 16 commits into from
Jan 13, 2025

Conversation

ISKonst
Copy link
Contributor

@ISKonst ISKonst commented Jan 2, 2025

Make sure that:

  • You have read the contribution guidelines.
  • You submit test cases (unit or integration tests) that back your changes.
  • Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

@ISKonst ISKonst changed the title refactor plugin-jwt: SPI extension for parsing JWT payloads to ensure compatibility with older versions of JWT tools [type:refactor] plugin-jwt SPI extension for parsing JWT payloads to ensure compatibility with older versions of JWT tools Jan 2, 2025
@moremind moremind added this to the 2.7.1 milestone Jan 2, 2025
@ISKonst
Copy link
Contributor Author

ISKonst commented Jan 5, 2025

The field handleType extended by plugin-jwt needs to be added to the database table.

@Aias00
Copy link
Contributor

Aias00 commented Jan 6, 2025

The field handleType extended by plugin-jwt needs to be added to the database table.

hi, just modify the sql, add alter sql

@ISKonst
Copy link
Contributor Author

ISKonst commented Jan 6, 2025 via email

@Aias00
Copy link
Contributor

Aias00 commented Jan 6, 2025

I know how to add new table fields in the initialization script (db/init). How can I add new table fields in the upgrade script (db/upgrade)? SKonst @.***  

------------------ 原始邮件 ------------------ 发件人: "apache/shenyu" @.>; 发送时间: 2025年1月6日(星期一) 下午2:11 @.>; @.@.>; 主题: Re: [apache/shenyu] [type:refactor] plugin-jwt SPI extension for parsing JWT payloads to ensure compatibility with older versions of JWT tools (PR #5882) The field handleType extended by plugin-jwt needs to be added to the database table. hi, just modify the sql, add alter sql — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you authored the thread.Message ID: @.***>

example:
ALTER TABLE dashboard_user ADD COLUMN client_id varchar(32) NULL DEFAULT NULL COMMENT 'client id';

@ISKonst
Copy link
Contributor Author

ISKonst commented Jan 8, 2025 via email

ISKonst and others added 2 commits January 8, 2025 09:35
…e added to the database table.

Upgrade database initialization script and upgrade script
@yu199195
Copy link
Member

goods to me.

@yu199195 yu199195 merged commit 8ca7f57 into apache:master Jan 13, 2025
44 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants