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

Remove download by downloadId functionality #1820

Merged

Conversation

avgustinmm
Copy link
Contributor

@avgustinmm avgustinmm commented Aug 14, 2024

This functionallity seems to get via AMQP (after some authentication)
a private (wihtout need of authentication) url to an artifact assigned
to the controller.

By default, DDI or DMF shall provide proper urls (for direct download)
to devices and if they have to be without authentication this shall be
solved in different ways - for instance separate download server providing
dedicated private / signed urls.

This functinallity is not a real hawkBit part but more like something
intended to solve some edge cases.
Since it is complicated, heeds support, doesn't solve wide spread use
cases, and could be achieved with other means - better to be removed.

@hawkbit-bot
Copy link

Can one of the admins verify this patch?

@avgustinmm avgustinmm requested a review from strailov August 14, 2024 12:31
This functionallity seems to get via AMQP (after some authentication)
a private (wihtout need of authentication) url to an artifact assigned
to the controller.

By default, DDI or DMF shall provide proper urls (for direct download)
to devices and if they have to be without authentication this shall be
solved in different ways - for instance separate download server providing
dedicated private / signed urls.

This functinallity is not a real hawkBit part but more like something
intended to solve some edge cases.
Since it is complicated, heeds support, doesn't solve wide spread use
cases, and could be achieved with other means - better to be removed.

Signed-off-by: Marinov Avgustin <[email protected]>
@avgustinmm avgustinmm force-pushed the maintenance/remove_download_server branch from 6f9eaba to 2868e24 Compare August 14, 2024 13:57
@avgustinmm avgustinmm changed the title Remove Download Server Remove download by downloadId functionality Aug 14, 2024
@avgustinmm avgustinmm merged commit d958d8e into eclipse-hawkbit:master Aug 14, 2024
2 checks passed
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.

2 participants