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

make doq transport settings more consistent with RFC #1682

Merged
merged 1 commit into from
Apr 10, 2022

Conversation

bluejekyll
Copy link
Member

@bluejekyll bluejekyll commented Apr 8, 2022

I'm trying to debug issues with DoQ and some servers out there, but can't get to the root yet. These changes are things that appear to differ from RFC and some inconsistency between the client and server.

related: #1687

@codecov
Copy link

codecov bot commented Apr 8, 2022

Codecov Report

Merging #1682 (a983bf3) into main (05f9642) will increase coverage by 1.03%.
The diff coverage is 91.30%.

❗ Current head a983bf3 differs from pull request most recent head 85a4faf. Consider uploading reports for the commit 85a4faf to get more accurate results

@@            Coverage Diff             @@
##             main    #1682      +/-   ##
==========================================
+ Coverage   78.84%   79.87%   +1.03%     
==========================================
  Files         182      184       +2     
  Lines       18824    18606     -218     
==========================================
+ Hits        14840    14860      +20     
+ Misses       3984     3746     -238     

@bluejekyll bluejekyll merged commit 5815504 into main Apr 10, 2022
@bluejekyll bluejekyll deleted the better-doq-rfc-compliance branch April 10, 2022 20:05
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.

None yet

1 participant