-
Notifications
You must be signed in to change notification settings - Fork 370
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
Csm merge from main #2130
Csm merge from main #2130
Conversation
Supports keeping post data without authentication. Relax TCP times. Add scheme to attributes. Signed-off-by: Achim Kraus <[email protected]>
Signed-off-by: Achim Kraus <[email protected]>
Signed-off-by: Achim Kraus <[email protected]>
Signed-off-by: Achim Kraus <[email protected]>
Add getRemoteSocketAddress to Exchange and use that. fixes: issue 2093 Signed-off-by: Achim Kraus <[email protected]>
Signed-off-by: Achim Kraus <[email protected]>
Most enhancement issues are closed after no activity. Mark them with hibernate makes it easier the select them again. Signed-off-by: Achim Kraus <[email protected]>
Signed-off-by: Achim Kraus <[email protected]>
The function requires much CPU load. Replaced it by OptionSet.hasOscore() and single call to OptionSet.asSortedList(). Signed-off-by: Achim Kraus <[email protected]>
Clear CustomOptionNumberRegistry after tests. Signed-off-by: Achim Kraus <[email protected]>
Extend LimitedRunable instead of passing Runnable to executeInBounds. Add missing datagramFilter.onDrop calls.
Add SPDX-License-Identifier: EPL-2.0 OR BSD-3-Clause Signed-off-by: Achim Kraus <[email protected]>
Signed-off-by: Achim Kraus <[email protected]>
Signed-off-by: Achim Kraus <[email protected]>
Signed-off-by: Achim Kraus <[email protected]>
Signed-off-by: Achim Kraus <[email protected]>
Signed-off-by: Achim Kraus <[email protected]>
Signed-off-by: Achim Kraus <[email protected]>
Signed-off-by: Achim Kraus <[email protected]>
Signed-off-by: Achim Kraus <[email protected]>
Also add more unit tests for MultiPskFileStore. Signed-off-by: Achim Kraus <[email protected]>
Signed-off-by: Achim Kraus <[email protected]>
Signed-off-by: Achim Kraus <[email protected]>
Add direct link to the form to report a new security issue in GitHub security advisories and explain how to access that form. The goal is to make reporting even easier. Signed-off-by: Marta Rybczynska <[email protected]>
Introduce a OptionDefinition to support custom options. Add a message code aware OptionRegistry to enable future TCP support. Deprecate a couple of old APIs, which are mainly moved to the new OptionDefinition. Signed-off-by: Achim Kraus <[email protected]>
Signed-off-by: Achim Kraus <[email protected]>
Signed-off-by: Achim Kraus <[email protected]>
Signed-off-by: Achim Kraus <[email protected]>
Signed-off-by: Achim Kraus <[email protected]>
Add tests for RawPublicKey (RFC7250). Signed-off-by: Achim Kraus <[email protected]>
Cancel all observation relations on send errors. Fix benchmark client. use debug instead of error for "too early responses". Add caching of principal and session-id to TLS. Update netty.io to 4.1.87.Final. Signed-off-by: Achim Kraus <[email protected]>
Signed-off-by: Achim Kraus <[email protected]>
Disabling the reuse of tokens for blockwise broke this function. Signed-off-by: Achim Kraus <[email protected]>
Signed-off-by: Achim Kraus <[email protected]>
Signed-off-by: Achim Kraus <[email protected]>
Signed-off-by: Achim Kraus <[email protected]>
Signed-off-by: Achim Kraus <[email protected]>
Add logging message, if the handshake failure may be caused by a wrong PSK secret. Issue: eclipse-californium#2109 Signed-off-by: Achim Kraus <[email protected]>
Issue eclipse-californium#2119 Signed-off-by: Achim Kraus <[email protected]>
Ignore API difference in enum-order for CipherSuite. Add interoperability tests. Signed-off-by: Achim Kraus <[email protected]>
Issue eclipse-californium#2127 Signed-off-by: Achim Kraus <[email protected]>
Signed-off-by: Rogier Cobben <[email protected]>
Thanks for your contribution. From the discussion in issue #2092 I'm not sure, what the right strategy should be. As I already wrote, if we agree to put it on a feature branch, we need a shepherd for that branch. |
It's an update to actual state of main. Either way it's a no regret IMHO. |
I don´t understand. The merge request concerns an update of csm branch itself. It does not touch main. This has no relation to the discussion whether csm branch gets merged into main. |
If the csm branch is considered to be developed for a certain time on it's own, then in my opinion the "shepherd" will be responsible for that merge. As you may have noticed, the author of that branch didn't agree on maintaining the work as branch. Without that "shepherd", I don't know, which committer is going to merge it. Sorry, I know in an other case you asked first about contribution (at that time to main), and I answered it's easier for me to have a PR. This time the point is, you don't ask me, you need to ask someone else. |
I didn't read that. Many is said in #2092 and it seems the discussion got derailed somehow. But, what I read is a lot of agreement and:
So the problem seems not to work in a branch, but to get the tcp feature usable in Leshan. A solution might be to "release" a californium version built from the branch to Maven Central, e.g. as "3.8.0-TCP-M1". This way Leshan (and Mule CoAP Connector) could benefit from a rfc8323 complient version and the tcp feature can be (alfa/beta) tested without risk to others. In the meantime main can be kept stable and merges of (parts of) changes can be done when considered stable enough. With the intention to get all merged, but paced by the time available to contributors. This PR concerned - I can't imagine any future "shepard" would object to a branch that´s not too diverged from main. Also, if the branch isn 't expected to get released soon there is little risk in merging, IMHO. That I should ask a non-existent "shepard" to merge, feels a bit Kafkaesque. The author seems not to object:
I ´m only able to make small contributions. But the idea of open-source is to combine (small) contributions to get the project, forward, isn't? |
Sure. It's also Kafkaesque to be blamed to run this project as a "one man show". But if it comes to work, only one man is left. As you may have noticed, I changed the job last year. Not that staying at that job would have preserved my time for working in Californium, the opposite is the case. Anyway, I still need to ramp down my efforts. The branch csm fails to build on my machine. It comes without documentation. the API breaks need to be escaped for the revapi checks. E.g. the csm branch breaks the TLS support. TLS was not considered for the first work, but I'm not sure, if that is considered to be acceptable or will be fixed or whatever is the intention. But again also here: I need to ramp down my efforts. If others want to jump in, welcome, if not, I don't see, that I can spend my time to shape the csm branch in order to fit common quality standard. We may also lower these standards, but in difference to the past years, I'm not the one who jumps then in to fix it. Overall: The last question from my side would then just be: |
A humble way of looking at this situation would be to ask yourself why you are the only one man left instead of denied the work of others. |
There may be many different views on that, I don't share yours. |
Sorry again, that you spend your time before asking. |
@boaks, really ...
It just means, "you can take my code as base if you want to move forward on this"...
Am I the only one who doesn't understand this ? So you are the project lead and the only one active committer. I just answer to this topic because you triggered me with : " It's also Kafkaesque to be blamed to run this project as a "one man show". But if it comes to work, only one man is left." which clearly target me and totally denies my works....
A CQ for a merge request between 2 branch already in Californium repository ? What would be the purpose of this ? @rogierc good luck for your collaboration on this with @boaks. |
As I wrote already last year, December: "at least I don't plan to spend my time in maintaining that." And though I'm not aware, that any committer will take care of it, I informed the contributor about that. |
Now that Californium has no corporate sponsors to improve and maintain such a complicated project and communication issues between committers, maybe it's time to move to maintenance mode:
I started to work with Californium 3.7 recently, debugging minor issues, and my feeling is that the code is too old, too complex, with an enormous scope (from base CoAP/DTLS lib to k8s clustering), and it's going to be challenging to keep running the project as-it-is without a clear full-time corporate workforce. |
Would it be OK to copy your comment to an new issue? |
Here you go: #2131 |
After all, it seems that none is interested in spending the time in being a shepherd for this work. |
Merge head of main into csm branch. Conflicts resolved.