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

hello world #1481

Closed
wants to merge 40 commits into from
Closed

hello world #1481

wants to merge 40 commits into from

Conversation

YoDaMa
Copy link
Contributor

@YoDaMa YoDaMa commented May 12, 2022

No description provided.

vishnureddy17 and others added 30 commits September 24, 2021 08:15
Add note to README about vNext discussions
Fixed typo in the React code example.
* Refined Topic Alias support. (Implement #1300)

Add automatic topic alias management functionality.
- On PUBLISH sending, the client can automatic using/assin Topic
Alias (optional).
- On PUBLISH receiving, the topic parameter of on message handler is
automatically complemented but the packet.topic preserves the original
topic.

Fix invalid tests.

* Fix typo.

* Fix comment.

* Rename the function name from `removeTopicAlias` to `removeTopicAliasAndRecoverTopicName`.

Add comments for caller side of `removeTopicAliasAndRecoverTopicName`.

* Captalize label.
#1333)

* Set noImplicitAny to false in test/typescript/tsconfig.json due to a compilation error resolving ws types
OnConnectCallback receives back a Connack packet as stated in the MQTT specs

* Add @types/ws in devDependencies
Restore noImplicitAny typescript rule
* fix: messageeId

* Fix messageId allocate twice on deliver.

resubscribe is out of MQTT spec. It is MQTT.js expansion.
On connect sequence, the following three steps are defined by the MQTT Spec.

1. The client sends CONNECT to the broker with CleanStart:false
2. The broker sends CONNACK to the client with SessionPresent:true if
session exists
3. The client re-sends in-flight PUBLISH and PUBREL messages

resubscribe was processed between the step 2 and step 3.
It's too early. The resubscribe might allocate messageId that is the
same as PUBLISH or PUBREL packet. It is not good.

So I moved resubscribe process to after the step 3.

* Removed invalid fallback code.

* Stored CONNACK packet instead of sessionPresent.

Co-authored-by: Yoseph Maguire <yoseph.maguire@gmail.com>
Co-authored-by: Caner Turkmen <caner@EcoGs-MacBook-Pro.local>
* Change examples to be more specific

* Fix ToC

* Fixed ToC order

Co-authored-by: Yoseph Maguire <yoseph.maguire@microsoft.com>
Version 8 is a stub, because in v8 types are integrated in the main package.
This avoids showing a warning when the broker address is just an IP
[DEP0123] DeprecationWarning: Setting the TLS ServerName to an IP address is not permitted by RFC 6066. This will be ignored in a future version.
Same fix as node-postgres has used brianc/node-postgres@d3c8eba
* fix: audit dev dependencies

* add changes

* timeout

* add timeout

* remove aftereach

* fixes
Co-authored-by: YuShifan <894402575bt@gmail.com>
* feat: add support for tls connection with alpn

* style: remove trailing comma from ALPNProtocols
* add draft of extended auth

* expose handleAuth for api

* do not handle auth packet directly for now

* handle auth packet

* point to dev version of mqtt-packet to make use of auth packages

* solve hacky connected/disconnected flagging to send auth packages

* remove unused code in handle auth

* add packetcallback to auth handler

* fix this and that issue

* point to mqtt-packet dev master

* switch to mqtt-packet 6.7.0 as our PR was merged

* merge

Co-authored-by: Lukas Läderach <lukas.laederach@3volutions.ch>
Co-authored-by: Cédric von Allmen <cedric.vonallmen@gmail.com>
Since number-allocator https://www.npmjs.com/package/number-allocator
1.0.9, js-sdsl https://www.npmjs.com/package/js-sdsl is used insteaad of
collections https://www.npmjs.com/package/collections.

collections modify intrinsic type such as Array, it has unexpected side
effect for users. So I removed the dependency of collections.

It is a partial fix for #1392.

Replacement for LRU map is still needed to remove collections dependency.
Process is enqueued if messageId allocation is failure.
update readme for announcements
YoDaMa and others added 10 commits February 7, 2022 09:52
* update CI

* Delete syncToDevOps.yml

* update yaml

* allow package lock updates

* add package lock

* updates

* fixes for testing

* fix typo

* remove eslint

* updates to testing

* update yaml

* continue on error

* remove eslint

* fixing ci issues
* bump version and update changelog
…tthub workflow (#1443)

* fix: fix regression from #1401 and allow CI test failures to break github workflow

* temporary rollback to verify break works

* put change back in

* add comment to fix

* fix misspelling
… is unavailable (#1445)

Co-authored-by: caoyawu <ucs_caoyawu@caoyawu>
Also remove an unused import in types/lib/client.d.ts.

Fixes #1408
@YoDaMa
Copy link
Contributor Author

YoDaMa commented May 12, 2022

mqttjs/mqttjs-v5#5

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.