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

SVCB overhaul #12

Merged
merged 12 commits into from
Jul 8, 2024
Merged

SVCB overhaul #12

merged 12 commits into from
Jul 8, 2024

Conversation

chrysn
Copy link
Member

@chrysn chrysn commented Jul 6, 2024

This moves (incomplete: will move) speculative content into just registering in the relevant registries (without, of course, retroactively making CoAP-over-TCP work that way automatically), and will also move content that was dropped in anr-bmbf-pivot/draft-lenders-core-dnr#6 in here.

@chrysn
Copy link
Member Author

chrysn commented Jul 6, 2024

The motto of the PR, as in the autors' latest video chat, was "more solid, less hätt'-I-war-I" (which is German roughly for coulda-shoulda-woulda)

Christian Amsüss added 10 commits July 7, 2024 21:39
This should have been "discovered service to URI", but it turned out to
be easier to reuse the URI construction defined in CoAP.
While processing material from
anr-bmbf-pivot/draft-lenders-core-dnr#6, this
appears to be more suitable (and it was an open item anyway).
Moving content in from anr-bmbf-pivot/draft-lenders-core-dnr#6 with a bit of simplification
This has been replaced with the actual description of how to use SVCB
records for CoAP (albeit only in an opportunistic or application
prescribed fashion)
@chrysn chrysn marked this pull request as ready for review July 8, 2024 22:19
@chrysn chrysn merged commit fdf82f0 into master Jul 8, 2024
2 checks passed
@chrysn chrysn deleted the svcb-overhaul branch July 8, 2024 22:19
@chrysn chrysn mentioned this pull request Jul 8, 2024
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.

1 participant