[Feature] Reuse weaver.url to define process locations #152
Labels
feature/db
Related to database or datatype manipulation.
triage/bug
Something isn't working
triage/enhancement
New feature or request
triage/feature
New requested feature.
Context
Until at least
weaver 1.8
, processes are stored with the 3 displayed fields below, directly using theweaver.url
value. While this is acceptable and works for base use case, it causes problems if the server configuration that usesweaver
is modified for any reason as all these locations become invalid.Processes deployed before the server reconfiguration would all point to invalid locations, meaning they cannot be correctly executed nor report the correct references in the response bodies.
For example, starting two server configurations from scratch that specify different
weaver.url
setting, the results are are follows after deploying some process. This is not the case ofbuiltin
processes which are redeployed on eachweaver
startup.Todo
Since
weaver
does not require to have the full URL in those fields, corresponding relative 'path' should be stored instead of the full URL. TheProcess
properties should me modified to provide a getter that combinesweaver.url
with the corresponding 'path' stored in db.weaver/weaver/datatype.py
Lines 673 to 686 in 5e2aa56
Since these path can also refer to a remote provider locations, the paths must not be assumed to only start with the
/processes
.The text was updated successfully, but these errors were encountered: