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

Add handler for ./well-known/core #30

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Add handler for ./well-known/core #30

wants to merge 2 commits into from

Conversation

JKRhb
Copy link
Owner

@JKRhb JKRhb commented Mar 16, 2021

This PR will implement a handler for /.well-known/core to support resource discovery using the CoRE Link Format (see RFC 6690). Once it is finished, it will resolve #2.

So far, only the available (unique) URLs are included in the payload that is returned by the handler for /.well-known/core as the setting of like ct or rt probably needs a way to defined resources on a "global" level so that the definitions in the coap in nodes don't contradict each other. I will have a deeper look into this while investigating the possibilities for implementing this feature.

@JKRhb JKRhb self-assigned this Mar 16, 2021
@JKRhb JKRhb force-pushed the well-known-core branch from ac764c9 to 887f4e5 Compare July 4, 2024 18:25
@coveralls
Copy link

coveralls commented Jul 4, 2024

Pull Request Test Coverage Report for Build 9798894676

Details

  • 4 of 9 (44.44%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-1.7%) to 80.137%

Changes Missing Coverage Covered Lines Changed/Added Lines %
coap/coap-in.js 4 9 44.44%
Totals Coverage Status
Change from base Build 9794631181: -1.7%
Covered Lines: 158
Relevant Lines: 190

💛 - Coveralls

1 similar comment
@coveralls
Copy link

Pull Request Test Coverage Report for Build 9798894676

Details

  • 4 of 9 (44.44%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-1.7%) to 80.137%

Changes Missing Coverage Covered Lines Changed/Added Lines %
coap/coap-in.js 4 9 44.44%
Totals Coverage Status
Change from base Build 9794631181: -1.7%
Covered Lines: 158
Relevant Lines: 190

💛 - Coveralls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement .well-known/core option for the coap server node
2 participants