Page MenuHomePhabricator
Differential D12437

[lib][web][native] Send CSAT in getKeyserverOverrideForAnInviteLink
ClosedPublic
Actions

Authored by bartek on Jun 13 2024, 11:40 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Dec 19, 5:01 AM
Unknown Object (File)
Thu, Dec 19, 5:01 AM
Unknown Object (File)
Thu, Dec 19, 5:00 AM
Unknown Object (File)
Thu, Dec 19, 5:00 AM
Unknown Object (File)
Wed, Nov 27, 5:20 PM
Unknown Object (File)
Nov 23 2024, 10:30 PM
Unknown Object (File)
Nov 23 2024, 9:00 PM
Unknown Object (File)
Nov 15 2024, 11:06 AM
View All 47 Files
Subscribers
ashoat
ginsu

Details

Reviewers
tomek
Commits
rCOMM52de63b4fcc5: [lib][web][native] Send CSAT in getKeyserverOverrideForAnInviteLink
Summary

getKeyserverOverrideForAnInviteLink() performs a Blob Service request but doesn't send Authorization header.
Followed multimedia fetching code and added optional AuthMetadata that is sent if usingCSAT == true.

Test Plan
  • On my local keyserver, created a community and created invite link
  • Opened the link in the browser - invitation was successful.
  • Searched Blob Service debug logs - Authorization header was present.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

bartek created this revision.Jun 13 2024, 11:40 PM
bartek held this revision as a draft.
Herald added a subscriber: ashoat. · View Herald TranscriptJun 13 2024, 11:40 PM
Harbormaster completed remote builds in B29679: Diff 41316.Jun 13 2024, 11:57 PM
bartek published this revision for review.Jun 14 2024, 3:00 AM
bartek edited the test plan for this revision. (Show Details)
tomek accepted this revision.Jun 14 2024, 5:36 AM
This revision is now accepted and ready to land.Jun 14 2024, 5:36 AM
ashoat added a subscriber: ginsu.Jun 14 2024, 2:30 PM

cc @ginsu, as I know the Farcaster channel work is closely modeled after the invite link work, and might have the same problem

bartek mentioned this in D12065: [native] introduce AutoJoinCommunityHandler.Jun 16 2024, 10:34 PM
bartek added a parent revision: D12446: [native] Send CSAT when fetching blob service media.Jun 18 2024, 12:33 AM
Closed by commit rCOMM52de63b4fcc5: [lib][web][native] Send CSAT in getKeyserverOverrideForAnInviteLink (authored by bartek). · Explain WhyJun 18 2024, 2:18 AM
This revision was automatically updated to reflect the committed changes.
bartek added a commit: rCOMM52de63b4fcc5: [lib][web][native] Send CSAT in getKeyserverOverrideForAnInviteLink.

Revision Contents
Changeset List

PathSize
lib/
shared/
7 lines
native/
navigation/
16 lines
web/
invite-links/
20 lines

Diff 41436

View Options

lib/shared/invite-links.js

Loading...
View Options

native/navigation/deep-links-context-provider.react.js

Loading...
View Options

web/invite-links/invite-link-handler.react.js

Loading...