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

update-bin-web-script-for-analytics #1937

Merged
merged 1 commit into from
Jun 23, 2023

Conversation

aprilrieger
Copy link
Collaborator

On setup of ATLA-Hyku, we noted that there was. failure in the web script making Google Analytics implementation fail with a PKCS12_parse: mac verify failure. On review of the prod-cred.p12 file on the server that is supposed to get created from the ./bin/worker script, but was an empty file.

Adding implementation of the ./bin/worker script which is different than the ./bin/web script, succeeded when ran in both the web and worker pods.

Sentry error: https://scientist-inc.sentry.io/issues/4092424464/?project=4505008236396544&referrer=project-issue-stream

Site where analytics is now working with the new implementation: https://demo.atla-hyku.notch8.cloud/works/7fdcfe46-6c9d-4eeb-8394-b7c236b363d3/stats?locale=en

CLICK ME Screenshot Screenshot 2023-04-15 at 00 54 52

On setup of ATLA-Hyku, we noted that there was. failure in the web script making Google Analytics implementation fail with a `PKCS12_parse: mac verify failure`. On review of the prod-cred.p12 file on the server that is supposed to get created from the ./bin/worker script, but was an empty file. 

Adding implementation of the ./bin/worker script which is different than the ./bin/web script, succeeded when ran in both the web and worker pods.

Sentry error: https://scientist-inc.sentry.io/issues/4092424464/?project=4505008236396544&referrer=project-issue-stream

Site where analytics is now working with the new implementation: https://demo.atla-hyku.notch8.cloud/works/7fdcfe46-6c9d-4eeb-8394-b7c236b363d3/stats?locale=en

<details><summary>CLICK ME Screenshot</summary>

</details>
@aprilrieger aprilrieger added the minor-ver for release notes label Apr 15, 2023
@orangewolf
Copy link
Member

this is a really long way to say "make worker and web bin scripts the same" =-)

@orangewolf orangewolf merged commit 50e28d7 into main Jun 23, 2023
@orangewolf orangewolf deleted the update-bin-web-script-for-analytics branch June 23, 2023 23:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
minor-ver for release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants