Skip to content

Conversation

@shreemaan-abhishek
Copy link
Contributor

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

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)

Signed-off-by: Abhishek Choudhary <shreemaan.abhishek@gmail.com>
…ate-deps

Signed-off-by: Abhishek Choudhary <shreemaan.abhishek@gmail.com>
Signed-off-by: Abhishek Choudhary <shreemaan.abhishek@gmail.com>
Signed-off-by: Abhishek Choudhary <shreemaan.abhishek@gmail.com>
Signed-off-by: Abhishek Choudhary <shreemaan.abhishek@gmail.com>
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
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
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
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
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
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.

GET /echo?args=%40%23%24%25%5E%26
--- response_body chomp
args=@#$%^&
args=@%23$%25%5E&
Copy link
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

Copy link
Member

@nic-6443 nic-6443 Jan 6, 2026

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
@shreemaan-abhishek shreemaan-abhishek merged commit fdfca68 into apache:master Jan 7, 2026
56 of 69 checks passed
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.

4 participants