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
F3356836: D12437.diff
Sat, Nov 23, 9:00 PM
Unknown Object (File)
Fri, Nov 15, 11:06 AM
Unknown Object (File)
Fri, Nov 15, 2:14 AM
Unknown Object (File)
Fri, Nov 15, 1:37 AM
Unknown Object (File)
Sun, Nov 10, 12:22 AM
Unknown Object (File)
Fri, Nov 1, 5:39 PM
Unknown Object (File)
Oct 13 2024, 6:36 AM
Unknown Object (File)
Oct 13 2024, 6:36 AM
View All 41 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
No Lint Coverage
Unit
No Test Coverage

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 41316

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...