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 4, 2024
1 parent 9af702a commit d3ef2d5
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 d3ef2d5

Please sign in to comment.