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.

Sign up for GitHub

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

Jump to bottom

sys/net/app/cord: Update to RD draft -27 #16112

Merged
merged 2 commits into from
Mar 1, 2021
Merged

sys/net/app/cord: Update to RD draft -27 #16112

merged 2 commits into from
Mar 1, 2021

Conversation

chrysn
Copy link
Member

@chrysn chrysn commented Mar 1, 2021

Contribution description

This enacts the one actual change since this was written (which is, simple registrations go to /.well-known/rd rather than /.well-known/core).

The remaining changes are just updating the references to -27 (updating sections and comments as needed). Unless the RFC editor changes anything about sections, the updated numbers can stay.

Testing procedure

  • Install latest aiocoap from git
  • Run ./aiocoap-rd
  • Run the examples/cord_epsim example
  • Watch aiocoap-rd's output: It does not warn about a client registering with the old .well-known/core name any more.

(but the code change is really minimal, so looking at the code change might be good enough.)

@chrysn chrysn added Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Area: CoAP Area: Constrained Application Protocol implementations labels Mar 1, 2021
@chrysn chrysn mentioned this pull request Mar 1, 2021
Merged
@benpicco benpicco merged commit b572e67 into RIOT-OS:master Mar 1, 2021
@chrysn chrysn deleted the cord-27 branch March 1, 2021 16:13
@kaspar030 kaspar030 added this to the Release 2021.04 milestone Apr 23, 2021
@chrysn chrysn added the Release notes: added Set on PRs that have been processed into the release notes for the current release. label Apr 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@benpicco benpicco benpicco approved these changes

@cgundogan cgundogan Awaiting requested review from cgundogan

@haukepetersen haukepetersen Awaiting requested review from haukepetersen

@jia200x jia200x Awaiting requested review from jia200x jia200x is a code owner

@miri64 miri64 Awaiting requested review from miri64 miri64 is a code owner

@PeterKietzmann PeterKietzmann Awaiting requested review from PeterKietzmann PeterKietzmann is a code owner

Assignees
No one assigned
Labels
Area: CoAP Area: Constrained Application Protocol implementations CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer Release notes: added Set on PRs that have been processed into the release notes for the current release. Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation
Projects
None yet
Milestone
Release 2021.04
Development

Successfully merging this pull request may close these issues.
3 participants
@chrysn @benpicco @kaspar030