Skip to content

PurchasesDiagnostics: don't test signature verification if it's disabled#2757

Merged
NachoSoto merged 2 commits into
mainfrom
diagnostics-signature-verification
Jul 6, 2023
Merged

PurchasesDiagnostics: don't test signature verification if it's disabled#2757
NachoSoto merged 2 commits into
mainfrom
diagnostics-signature-verification

Conversation

@NachoSoto

Copy link
Copy Markdown
Contributor

Signature verification is not working due to changing signature (see #2744), but the debug screen shouldn't show an error if verification isn't enabled.

@NachoSoto NachoSoto added the pr:fix A bug fix label Jul 6, 2023
@NachoSoto NachoSoto requested a review from a team July 6, 2023 17:42
@NachoSoto NachoSoto force-pushed the diagnostics-signature-verification branch from 03d7aab to 9aef63b Compare July 6, 2023 19:32
…abled

Signature verification is not working due to changing signature (see #2744), but the debug screen shouldn't show an error if verification isn't enabled.
@NachoSoto NachoSoto force-pushed the diagnostics-signature-verification branch from 9aef63b to 2faf4f9 Compare July 6, 2023 19:33
This will be re-enabled in #2744
@NachoSoto NachoSoto enabled auto-merge (squash) July 6, 2023 20:06
@NachoSoto NachoSoto merged commit 8392161 into main Jul 6, 2023
@NachoSoto NachoSoto deleted the diagnostics-signature-verification branch July 6, 2023 20:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr:fix A bug fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants