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

feat(uptime): Extract domain and suffix from urls when creating uptime monitors #76367

Merged
merged 2 commits into from
Sep 4, 2024

Conversation

wedamija
Copy link
Member

This uses tldextract to extract the domain and suffix from urls, for use in limiting number of monitors per domain, analytics and detecting anomalous failures across monitors.

Copy link

codecov bot commented Aug 26, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##           master   #76367   +/-   ##
=======================================
  Coverage   78.18%   78.19%           
=======================================
  Files        6896     6896           
  Lines      306552   306554    +2     
  Branches    50259    50259           
=======================================
+ Hits       239690   239696    +6     
+ Misses      60454    60452    -2     
+ Partials     6408     6406    -2     

Base automatically changed from danf/uptime-url-cols to master September 3, 2024 18:37
@wedamija wedamija force-pushed the danf/uptime-extract-domain-suffix branch from cd67e72 to 5f34a4e Compare September 3, 2024 18:59
…e monitors

This uses `tldextract` to extract the domain and suffix from urls, for use in limiting number of monitors per domain, analytics and detecting anomalous failures across monitors.
@wedamija wedamija force-pushed the danf/uptime-extract-domain-suffix branch from 5f34a4e to ff7be87 Compare September 3, 2024 23:37
@wedamija wedamija merged commit 4167b1f into master Sep 4, 2024
50 checks passed
@wedamija wedamija deleted the danf/uptime-extract-domain-suffix branch September 4, 2024 18:19
@github-actions github-actions bot locked and limited conversation to collaborators Sep 20, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Scope: Backend Automatically applied to PRs that change backend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants