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.

Sign up for GitHub

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

Jump to bottom

[APPC-4345] Android wallet authentication jwt management #1229

Merged
merged 2 commits into from
Jan 20, 2025
Merged

[APPC-4345] Android wallet authentication jwt management #1229

merged 2 commits into from
Jan 20, 2025

Conversation

ricardo-frango
Copy link
Contributor

@ricardo-frango ricardo-frango commented Dec 18, 2024 •
edited by jira bot
Loading

What does this PR do?

This PR creates the implementation to add the Authorization header with the new JWT for all endpoints. Also replaces the previous implementation using the EWT.

Database changed?

No

Where should the reviewer start?

  • BackendApiModule.kt
  • BaseHttpClient.kt
  • RenewJwtApi.kt
  • RenewJwtInterceptor.kt
  • JwtResponse.kt
  • SessionManager.kt
  • BaseApiModule.kt
  • EwtAuthenticatorService.kt
  • AdyenPaymentInteractor.kt
  • AmazonPayRepository.kt
  • CreateAmazonPayTransactionTopUpUseCase.kt
  • CreateAmazonPayTransactionUseCase.kt
  • CarrierBillingRepository.kt
  • GooglePayWebRepository.kt
  • PayPalV2Repository.kt
  • SandboxRepository.kt
  • CreateSandboxTransactionUseCase.kt
  • SandboxViewModel.kt
  • TrueLayerRepository.kt
  • VkPayRepository.kt
  • CreditsRemoteRepository.java
  • BdsBillingModule.kt
  • RegisterFirebaseTokenUseCaseImpl.kt
  • FirebaseMessagingAPI.kt
  • FirebaseMessagingRepository.kt
  • FirebaseMessagingRepositoryImpl.kt
  • GetInvoiceByIdUseCase.kt
  • DeleteWalletInteract.kt
  • DeleteCachedGuestWalletUseCase.kt
  • GetPaymentInfoNewCardModelUseCase.kt
  • GetStoredCardsUseCase.kt
  • GetPaymentInfoModelUseCase.kt
  • RedeemGiftRepository.kt
  • RedeemGiftUseCase.kt
  • CachedGuestWalletRepository.kt
  • TransactionsHistoryRepository.kt
  • GetPaymentInfoFilterByCardModelUseCase.kt
  • TransactionDetailsViewModel.kt
  • BackupLogApi.kt
  • CachedGuestWalletApi.kt
  • RedeemGiftApi.kt
  • SupportApi.kt
  • TransactionsApi.kt
  • WithdrawApi.kt
  • BackendApiModule.kt
  • EmailApi.kt
  • EmailRepository.kt
  • PostUserEmailUseCase.kt
  • JwtRepository.kt
  • JwtAuthenticatorService.kt
  • AdyenApi.kt
  • AdyenSessionApi.kt
  • AmazonPayApi.kt
  • BackupEmailApi.kt
  • BrokerBdsApi.kt
  • CarrierBillingApi.kt
  • PaypalV2Api.kt
  • SandboxApi.kt
  • TrueLayerApi.kt
  • VkPayApi.kt
  • InappBillingApi.kt
  • BrokerApiModule.kt
  • BackupRepository.kt
  • BackupSuccessLogUseCase.kt
  • SupportRepository.kt
  • BdsAppcoinsRewardsRepository.kt
  • RemoteRepository.kt
  • BdsRepository.kt
  • RemoteRepository.kt
  • BillingPaymentProofSubmissionImpl.kt
  • AdyenPaymentRepository.kt
  • AppcoinsBillingReceiverActivity.kt
  • BillingService.kt

How should this be manually tested?

Test all the app and search for eventual issues.

What are the relevant tickets?

Tickets related to this pull-request: APPC-4345
Tickets related to this pull-request: APPC-4346

Code Review Checklist

  • Architecture
  • Documentation on public interfaces
  • Database changed?
  • Database migration (if applicable)?
  • Remove comments & unused code & forgotten testing Logs
  • Codestyle
  • Functional QA tests pass

@ricardo-frango ricardo-frango force-pushed the feature/APPC-4345_android-wallet-authentication-jwt-management branch 2 times, most recently from da82234 to b7bf3ea Compare December 19, 2024 10:32
@rodolfobcardoso rodolfobcardoso added this to the 3.36 milestone Dec 20, 2024
@ricardo-frango ricardo-frango force-pushed the feature/APPC-4345_android-wallet-authentication-jwt-management branch from b7bf3ea to d10e161 Compare December 30, 2024 12:35
@ricardo-frango ricardo-frango force-pushed the feature/APPC-4345_android-wallet-authentication-jwt-management branch from d10e161 to cf1aa13 Compare December 30, 2024 14:10
@ricardo-frango
Copy link
Contributor Author

run-jenkins-build

@ricardo-frango
Copy link
Contributor Author

run-jenkins-build

2 similar comments
@ricardo-frango
Copy link
Contributor Author

run-jenkins-build

@ricardo-frango
Copy link
Contributor Author

run-jenkins-build

@rodolfobcardoso rodolfobcardoso merged commit 7ac3cb9 into dev Jan 20, 2025
2 checks passed
@rodolfobcardoso rodolfobcardoso deleted the feature/APPC-4345_android-wallet-authentication-jwt-management branch January 20, 2025 12:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rodolfobcardoso rodolfobcardoso rodolfobcardoso approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
3.36
Development

Successfully merging this pull request may close these issues.
2 participants
@ricardo-frango @rodolfobcardoso