Skip to content

Commit

Permalink
Correctly namespace the notify URLs
Browse files Browse the repository at this point in the history
These are supposed to be URLs down from the top of the service's URL
namespace; this means the initial `v1` is required.

It would be better to refactor so that the REST API and the notify API
were provided by the same URL definition.
  • Loading branch information
amrc-benmorrow committed Nov 14, 2024
1 parent 86f372a commit bee0fe1
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions acs-configdb/lib/notify.js
Original file line number Diff line number Diff line change
Expand Up @@ -190,15 +190,15 @@ export class CDBNotify extends Notify {
build_handlers () {
return [
new WatchFilter({
path: "app/:app/object/:obj",
path: "v1/app/:app/object/:obj",
handler: (s, a, o) => new CDBWatch(s, a).single_config(o),
}),
new WatchFilter({
path: "app/:app/object/",
path: "v1/app/:app/object/",
handler: (s, a) => new CDBWatch(s, a).config_list(),
}),
new SearchFilter({
path: "app/:app/object/",
path: "v1/app/:app/object/",
handler: (s, f, a) => new CDBWatch(s, a).config_search(f),
}),
];
Expand Down

0 comments on commit bee0fe1

Please sign in to comment.