Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support changing apiKey in session callback #1855

Merged
10 commits merged into from Jul 4, 2023

Conversation

ghost
Copy link

@ghost ghost commented May 31, 2023

Goal

Session has it's own apikey and can be reset in session callback.

Changeset

Added apiKey to session and keep the apikey in session store.

Testing

Unit tests and scenario test

@ghost ghost requested a review from lemnik May 31, 2023 12:55
@ghost ghost force-pushed the PLAT-4974_Support_change_of_API_key_in_session_callback branch from 80690c1 to dfa4ce6 Compare May 31, 2023 12:56
@bugsnagbot
Copy link
Collaborator

bugsnagbot commented May 31, 2023

Android notifier sizes

Format Size impact of Bugsnag (kB) Size impact of Bugsnag when Minified (kB)
APK 1921.69 1696.73
arm64_v8a 680.33 459.15
armeabi_v7a 614.8 393.62
x86 754.04 532.86
x86_64 725.38 504.19

Generated by 🚫 Danger

@ghost ghost force-pushed the PLAT-4974_Support_change_of_API_key_in_session_callback branch 7 times, most recently from 24f48e7 to c98a7e0 Compare May 31, 2023 16:46
@ghost ghost force-pushed the PLAT-4974_Support_change_of_API_key_in_session_callback branch 3 times, most recently from 335f66e to 97bf5c0 Compare June 5, 2023 15:20
@ghost ghost requested a review from lemnik June 5, 2023 15:51
@ghost ghost force-pushed the PLAT-4974_Support_change_of_API_key_in_session_callback branch 3 times, most recently from 7a66c9f to d2ac0c1 Compare June 6, 2023 15:07
@ghost ghost requested a review from lemnik June 6, 2023 15:18
CHANGELOG.md Outdated Show resolved Hide resolved
@ghost ghost force-pushed the PLAT-4974_Support_change_of_API_key_in_session_callback branch 3 times, most recently from 03a3f77 to 1fc471d Compare June 7, 2023 09:11
@lemnik lemnik changed the title Plat 4974 support change of api key in session callback Support changing apiKey in session callback Jun 9, 2023
@ghost ghost requested a review from lemnik June 9, 2023 08:35
CHANGELOG.md Outdated Show resolved Hide resolved
@ghost ghost force-pushed the PLAT-4974_Support_change_of_API_key_in_session_callback branch from 1fc471d to 27d7278 Compare June 15, 2023 14:16
@ghost ghost requested a review from lemnik June 15, 2023 14:18
@ghost ghost force-pushed the PLAT-4974_Support_change_of_API_key_in_session_callback branch 4 times, most recently from 6ef96eb to cf69006 Compare June 20, 2023 14:23
@ghost ghost requested a review from lemnik June 20, 2023 15:02
@ghost ghost force-pushed the PLAT-4974_Support_change_of_API_key_in_session_callback branch 3 times, most recently from b8f5026 to 07021dc Compare June 26, 2023 10:48
@ghost ghost requested a review from lemnik June 26, 2023 11:23
@ghost ghost force-pushed the PLAT-4974_Support_change_of_API_key_in_session_callback branch from 07021dc to 66dc291 Compare June 26, 2023 14:09
Copy link
Contributor

@lemnik lemnik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking great, with some tiny final requests 😄

@ghost ghost force-pushed the PLAT-4974_Support_change_of_API_key_in_session_callback branch from 66dc291 to 70a7bfc Compare June 29, 2023 08:22
@ghost ghost requested a review from lemnik June 29, 2023 08:23
Copy link
Contributor

@lemnik lemnik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - Pending CHANGELOG conflicts and a CI run

@ghost ghost merged commit aa5b60a into next Jul 4, 2023
28 checks passed
@ghost ghost deleted the PLAT-4974_Support_change_of_API_key_in_session_callback branch July 4, 2023 15:38
@ghost ghost mentioned this pull request Jul 25, 2023
This pull request was closed.
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.

None yet

3 participants