Closed kirillplatonov closed 2 months ago
cc @zzooeeyy @paulomarg
@matteodepalo please take a look at this one too. It's important upgrade to move forward with migration to id_token.
Hi @kirillplatonov, we're taking a look now.
@byrichardpowell made adjustments based on your feedback. Code become even simpler 👍
What this PR does
Switching from hardcoded AppBridge 3 version to AppBridge CDN. This removes the need to pass
params[:host]
forfullpage_redirect_to
helper and fixes redirect with token exchange auth. I manually tested it with both token exchange and legacy auth.Checklist
Before submitting the PR, please consider if any of the following are needed:
CHANGELOG.md
if the changes would impact usersREADME.md
, if appropriate./docs
, if necessary