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

Extract client name from the URL (DoH and DoT) #317

Merged
merged 2 commits into from
Oct 13, 2021
Merged

Conversation

0xERR0R
Copy link
Owner

@0xERR0R 0xERR0R commented Oct 13, 2021

No description provided.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@codecov
Copy link

codecov bot commented Oct 13, 2021

Codecov Report

Merging #317 (e135ff2) into development (c22292e) will increase coverage by 1.32%.
The diff coverage is 88.67%.

Impacted file tree graph

@@               Coverage Diff               @@
##           development     #317      +/-   ##
===============================================
+ Coverage        95.27%   96.60%   +1.32%     
===============================================
  Files               29       29              
  Lines             2160     2181      +21     
===============================================
+ Hits              2058     2107      +49     
+ Misses              74       49      -25     
+ Partials            28       25       -3     
Impacted Files Coverage Δ
server/server.go 90.59% <81.81%> (+10.17%) ⬆️
server/server_endpoints.go 95.20% <90.47%> (+6.69%) ⬆️
resolver/client_names_resolver.go 100.00% <100.00%> (ø)
resolver/resolver.go 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c22292e...e135ff2. Read the comment docs.

@0xERR0R 0xERR0R merged commit a90fb5d into development Oct 13, 2021
@delete-merged-branch delete-merged-branch bot deleted the fb-client-id branch October 13, 2021 20:47
@0xERR0R 0xERR0R linked an issue Oct 13, 2021 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Extract client name from the URL (DoH and DoT)
1 participant