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

catalog web proxy error for url http://www.google-analytics.com/collect #4508

Closed
FuhuXia opened this issue Oct 23, 2023 · 3 comments
Closed
Assignees
Labels
bug Software defect or bug

Comments

@FuhuXia
Copy link
Member

FuhuXia commented Oct 23, 2023

NewRelic Log review on prod:catalog-web and prod:catalog-admin showing these error. Staging space has similiar error.

urllib3.exceptions.MaxRetryError: HTTPSConnectionPool(host='proxy-gsa-datagov-prod-catalog-admin.apps.internal',
port=61443): Max retries exceeded with url: http://www.google-analytics.com/collect (Caused by ProxyError('Cannot
connect to proxy.', RemoteDisconnected('Remote end closed connection without response')))

The error stared on 2023-10-03 4:08PM EDT.

image

How to reproduce

Search MaxRetryError in the NewReloc

Sketch

[Notes or a checklist reflecting our understanding of the selected approach]

@FuhuXia FuhuXia added the bug Software defect or bug label Oct 23, 2023
@FuhuXia
Copy link
Member Author

FuhuXia commented Oct 23, 2023

According to ckanext-googleanalytics readme, it needs to send data to url www.google-analytics.com to send events data.

Adds Google Analytics Event Tracking to resource download links, so that resource downloads will be displayed as Events in the Google Analytics reporting interface.

Adds Google Analytics Event Tracking to some API calls so that usage of the API can be reported on via Google Analytics.

Add Google Analytics Event Tracking function that can be used in any exstension to create your custom events tracking.

So we need to add www.google-analytics.com to our proxy allow list.

@FuhuXia FuhuXia mentioned this issue Oct 23, 2023
4 tasks
@FuhuXia
Copy link
Member Author

FuhuXia commented Oct 23, 2023

We have added www.google-analytics.com to the allow list, see the PR above.

But in the ckanext-googleanalytics code, HTTP instead of HTTPS is used. We need convince ckan upstream to use HTTPS, in the meanwhile we can use a fork.

@Jin-Sun-tts
Copy link
Contributor

verified in the New Relic log, after fixes there is no more error message:

Screenshot 2023-10-25 at 4 28 05 PM

@hkdctol hkdctol moved this from ✔ Done to 🗄 Closed in data.gov team board Oct 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Software defect or bug
Projects
Archived in project
Development

No branches or pull requests

2 participants