Github user aledsage commented on the issue:
https://github.com/apache/brooklyn-server/pull/899
@ahgittin One thing I havenât done is to behave differently if
`?force=true` is used (to fail, rather than just returning a code that
indicates their request to add the bundle was ignored because it has been
forcibly removed). Do you think itâs worth adding that?
On a similar note, `CatalogResource.createFromArchive()` turns a
`IGNORING_BUNDLE_AREADY_INSTALLED` into a 400. The 400 feels wrong, but given
weâre explicitly doing that in the code then feels like we should probably
leave it and not change the behaviour.
Iâve *not* done the same for `IGNORING_BUNDLE_FORCIBLY_REMOVED`, so that
will return a 202 I think. That feels wrong - we didnât create anything.
Iâm tempted to change that to give a 400 as well. This would give the same
error whether or not `force` was supplied.
Thoughts?
---
