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

chore(deps): bump github.com/eclipse/paho.mqtt.golang from 1.3.5 to 1.4.1 in /test/e2e #1255

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Aug 22, 2022

Bumps github.com/eclipse/paho.mqtt.golang from 1.3.5 to 1.4.1.

Release notes

Sourced from github.com/eclipse/paho.mqtt.golang's releases.

v1.4.1

The license update in v1.4.0 included changes which prevented https://pkg.go.dev from displaying the documentation. The only change in this release is to update LICENSE such that it includes the full test of the Eclipse Public License v2.0 (tested this on a fork and it appears to resolve the issue).

v1.4.0

In the year since the release of v1.3.5 the majority of changes have been small incremental improvements/fixes. It is worth noting that the library has moved to the EPL v2.0 license (faq).

A few new features have been added:

  • Managing the connection to the broker can become involved (e.g. setting unusual TCP/TLS options, need for a non-standard TLS library, require a specific web-socket library etc) and the library already has too many options! To address this a new option, SetCustomOpenConectionFn, has been added that enables users to provide a function (func(uri *url.URL, options ClientOptions) (net.Conn, error)) that will be called to establish a connection (replacing the inbuilt code). This should enable edge cases to be addressed without the need for further changes to this library.
  • When connecting/reconnecting it is possible that there will be a lot of packets queued; in the past the library sent all of these packets as rapidly as possible when the connection came up. However this could saturate some network links causing the connection to drop (a vicious circle). The new option SetMaxResumePubInFlight allows a limit to be imposed; the library will send out the requested number of packets and then wait for acknowledgements before sending more (only applicable at QOS1+).

Thanks to everyone who submitted issues and contributed code (list of the main merged pull requests below):

What's Changed

New Contributors

Full Changelog: eclipse/paho.mqtt.golang@v1.3.5...v1.4.0

Commits
  • bcb58ee Update license to meet pkg.go.dev requirements
  • 8717376 Merge pull request #590 from ptsneves/pneves-disconnect-workaround
  • 2e83582 client.go Remove Disconnect()'s wait on c.commsStopped.
  • eaac59b Merge pull request #586 from ptsneves/disconnect-refactor
  • ac29fdf client: localized disconnectSent logic
  • 3ec7db3 client: refactor setConnected(disconnected) out of conditional
  • 7254d2c client: set disconnect as defer
  • 2d3b04d Merge pull request #577 from ChIoT-Tech/master
  • 721bf9f If c.options.Dialer was nil this caused a panic.
  • 44f71c7 Merge pull request #575 from GilGil1/master
  • Additional commits viewable in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

@codecov-commenter
Copy link

Codecov Report

Merging #1255 (ff9d6ec) into master (530ce52) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1255   +/-   ##
=======================================
  Coverage   42.72%   42.72%           
=======================================
  Files          73       73           
  Lines        6486     6486           
=======================================
  Hits         2771     2771           
  Misses       3418     3418           
  Partials      297      297           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Bumps [github.com/eclipse/paho.mqtt.golang](https://github.com/eclipse/paho.mqtt.golang) from 1.3.5 to 1.4.1.
- [Release notes](https://github.com/eclipse/paho.mqtt.golang/releases)
- [Commits](eclipse/paho.mqtt.golang@v1.3.5...v1.4.1)

---
updated-dependencies:
- dependency-name: github.com/eclipse/paho.mqtt.golang
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot bot force-pushed the dependabot/go_modules/test/e2e/github.com/eclipse/paho.mqtt.golang-1.4.1 branch from ff9d6ec to 701da38 Compare August 23, 2022 01:40
@tao12345666333 tao12345666333 merged commit c4b04b3 into master Aug 29, 2022
@tao12345666333 tao12345666333 deleted the dependabot/go_modules/test/e2e/github.com/eclipse/paho.mqtt.golang-1.4.1 branch August 29, 2022 08:16
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.

None yet

3 participants