Skip to content

fix(realtime): Send version when joining channel and remove jwt check#1166

Merged
grdsdev merged 1 commit into
mainfrom
guilherme/clibs-99-supabase-flutter-remove-jwt-check-and-send-x-client-info
May 6, 2025
Merged

fix(realtime): Send version when joining channel and remove jwt check#1166
grdsdev merged 1 commit into
mainfrom
guilherme/clibs-99-supabase-flutter-remove-jwt-check-and-send-x-client-info

Conversation

@grdsdev

@grdsdev grdsdev commented May 6, 2025

Copy link
Copy Markdown
Contributor

What is the new behavior?

  • Remove the jwt check when pushing access token
  • On set auth, also send the client info for better internal reports.

Additional context

supabase/realtime-js#451
supabase/realtime-js#455

@grdsdev grdsdev requested review from Vinzent03 and dshukertjr May 6, 2025 18:22
@coveralls

Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 14866917445

Details

  • 3 of 3 (100.0%) changed or added relevant lines in 1 file are covered.
  • 5 unchanged lines in 1 file lost coverage.
  • Overall coverage decreased (-0.07%) to 75.328%

Files with Coverage Reduction New Missed Lines %
packages/realtime_client/lib/src/realtime_channel.dart 5 66.78%
Totals Coverage Status
Change from base Build 14844996713: -0.07%
Covered Lines: 2870
Relevant Lines: 3810

💛 - Coveralls

@grdsdev grdsdev merged commit 9ccd890 into main May 6, 2025
@grdsdev grdsdev deleted the guilherme/clibs-99-supabase-flutter-remove-jwt-check-and-send-x-client-info branch May 6, 2025 19:46
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.

3 participants