search

rebus-org / Rebus

:bus: Simple and lean service bus implementation for .NET
https://mookid.dk/category/rebus
Other
2.26k stars 353 forks source link

#1157 add net 6 so dependencies can be decreased #1159

Closed thompson-tomo closed 2 months ago

thompson-tomo commented 2 months ago

Rebus is MIT-licensed. The code submitted in this pull request needs to carry the MIT license too. By leaving this text in, I hereby acknowledge that the code submitted in the pull request has the MIT license and can be merged with the Rebus codebase.

Closes #1157

CLAassistant commented 2 months ago

CLA assistant check
All committers have signed the CLA.

mookid8000 commented 2 months ago

Hi @thompson-tomo , thanks for contributing 🙂 I am not sure I understand what this is about...? I mean, I can see what you changed, but I fail to understand what will be possible that currently isn't....?

thompson-tomo commented 2 months ago

The biggest change is the fact the package is now compiled for net 6. The benefit is that release will be smaller as less dependencies & at the same time if a package like STJ was impacted by a CVE the application's using rebus would also be impacted.

mookid8000 commented 2 months ago

What's STJ and a CVE?

thompson-tomo commented 2 months ago

STJ is the system.text.json dependency and a CVE Is a security topic -> https://www.redhat.com/en/topics/security/what-is-cve?darkschemeovr=1

mookid8000 commented 2 months ago

Ah, I see! I'll merge your PR now then

mookid8000 commented 2 months ago

It's out as Rebus 8.4.0 on NUGet.org now 🙂 thanks for contributing!

thompson-tomo commented 2 months ago

No problem at all. 🙂