databendlabs / databend

๐——๐—ฎ๐˜๐—ฎ, ๐—”๐—ป๐—ฎ๐—น๐˜†๐˜๐—ถ๐—ฐ๐˜€ & ๐—”๐—œ. Modern alternative to Snowflake. Cost-effective and simple for massive-scale analytics. https://databend.com
https://docs.databend.com
Other
7.88k stars 753 forks source link

fix: token and cookie session id miss match. #16786

Closed youngsofun closed 2 weeks ago

youngsofun commented 2 weeks ago

I hereby agree to the terms of the CLA available at: https://docs.databend.com/dev/policies/cla/

Summary

related feature not in use yet, so the pr is safe. find something need improvement when working with the clients.

  1. session token use the session id generated when processing cookie and stored in context
  2. small refactor of request login and refresh

Tests

Type of change


This change isโ€‚Reviewable

youngsofun commented 2 weeks ago

cc @sundy-li ready to review