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

MQTT protocol binding is wrong #25

Open
hsebs opened this issue Oct 4, 2019 · 2 comments
Open

MQTT protocol binding is wrong #25

hsebs opened this issue Oct 4, 2019 · 2 comments

Comments

@hsebs
Copy link

hsebs commented Oct 4, 2019

delete jsonObj.csr.srv;

jsonObj of retrieve_CSEBase_mqtt is wrong.
according to TR-0051, csr has srv and it does not have srt.
But it deletes jsonObj.csr.srv instead of jsonObj.csr.srt.

It causes bad filed error when Mobius (or nCube-Rosemary) works as mn
{"m2m:dbg":"ER_BAD_FIELD_ERROR: Unknown column 'srt' in 'field list'"}

hsebs added a commit to hsebs/Mobius that referenced this issue Oct 4, 2019
@hsebs hsebs mentioned this issue Oct 4, 2019
@hsebs
Copy link
Author

hsebs commented Oct 4, 2019

var reg_req_topic = util.format('/oneM2M/reg_req/%s/%s/%s', usecseid.replace('/', ':'), cseid.replace('/', ':'), request.headers.bodytype);

var reg_req_topic = util.format('/oneM2M/reg_req/%s/%s/%s', usecseid.replace('/', ':'), cseid.replace('/', ':'), request.headers.bodytype);

var forward_topic = util.format('/oneM2M/req/%s/%s', usecseid.replace('/', ':'), forward_cseid);

reg_req_topic of mqtt_app is wrong.
according to TS-0010, leading '/'s of originator have to be omitted when it does not Credential-ID.
But it replaces leading '/'s with ':' even if originator is not Credential-ID. (it uses CSE-ID)
Receiver can't be Credential-ID, but it replaces leading '/'s with ':' of receiver.

@hsebs
Copy link
Author

hsebs commented Oct 4, 2019

var resp_topic = util.format('/oneM2M/resp/%s/#', usecseid.replace('/', ''));

var req_topic = util.format('/oneM2M/req/+/%s/+', usecseid.replace('/', ''));

var reg_req_topic = util.format('/oneM2M/reg_req/+/%s/+', usecseid.replace('/', ''));

topics of subscribe are wrong.
according to TS-0010, non-leading '/'s of originator and receiver have to be replaced with ':'.
But it omits non-leading '/'s of originator and receiver.

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

No branches or pull requests

1 participant