Skip to content

chore: update dependencies#12862

Merged
shreemaan-abhishek merged 5 commits into
apache:masterfrom
shreemaan-abhishek:sabc/chore/update-deps
Jan 7, 2026
Merged

chore: update dependencies#12862
shreemaan-abhishek merged 5 commits into
apache:masterfrom
shreemaan-abhishek:sabc/chore/update-deps

Conversation

@shreemaan-abhishek

@shreemaan-abhishek shreemaan-abhishek commented Jan 5, 2026

Copy link
Copy Markdown
Contributor

Description

Upgrade lua-resty-openidc, lua-resty-session, lua-resty-rocketmq and net-url.

All of these libraries have new latest updates, bugfixes and security enhancements. Specially lua-resty-openidc and lua-resty-session.

Security: Enforces AES-256-GCM encryption by default (replacing HMAC-signed plaintext) for full data confidentiality.
Efficiency: Enables automatic compression (Deflate) for payloads >1KB, significantly reducing cookie size and fragmentation overhead.

Checklist

  • I have explained the need for this PR and the problem it solves
  • I have explained the changes or the new features added to this PR
  • I have added tests corresponding to this change
  • I have updated the documentation to reflect this change
  • I have verified that this change is backward compatible (If not, please discuss on the APISIX mailing list first)

Fixes #9306

Signed-off-by: Abhishek Choudhary <[email protected]>
Signed-off-by: Abhishek Choudhary <[email protected]>
Signed-off-by: Abhishek Choudhary <[email protected]>
Signed-off-by: Abhishek Choudhary <[email protected]>
function _M.access(conf, ctx)
local current_uri = ctx.var.uri
local session_obj_read, session_present = session.open()
local session_obj, sess_err, session_present = session.open()

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

the response format has changed in the new version

}
session_obj_write:start()
session_obj_write.data.access_token = access_token
session_obj_write:open()

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

:start has been removed in the new version

core.log.info("rocketmq nameserver_list[1] port ",
prod.client.nameservers[1].port)
core.log.info("rocketmq nameserver_list[1]: ",
prod.client.nameservers[1])

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

port is no longer available in this data structure.

-- Extract cookie which is not authenticated
local cookie_str = concatenate_cookies(res.headers['Set-Cookie'])
-- set a random cookie
local cookie_str = "foobaar"

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

the previous session library used by lua-resty-openidc would set Set-Cookie header. This no longer happens in the new version.


local cookie_str = concatenate_cookies(res.headers['Set-Cookie'])
local parts = {}
for part in string.gmatch(cookie_str, "[^|]+") do

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

cookies are no longer | separated, they are in fact encrypted in the new version.

Comment thread t/plugin/serverless.t
GET /echo?args=%40%23%24%25%5E%26
--- response_body chomp
args=@#$%^&
args=@%23$%25%5E&

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

change due to net-url upgrade

@nic-6443 nic-6443 Jan 6, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

In other words, the three characters # / % / ^ are no longer decoded by default, which may cause compatibility issues. However, as long as the new behavior conforms to URL parser standards, it should also be upgraded.

@shreemaan-abhishek
shreemaan-abhishek marked this pull request as ready for review January 6, 2026 06:21
@dosubot dosubot Bot added size:M This PR changes 30-99 lines, ignoring generated files. dependencies Pull requests that update a dependency file labels Jan 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug: openid-connect plugin - existing session randomly unavailable for introspection / token renewal

4 participants