Skip to content

[Notifications] Strip trailing slash for fallback URL path

Kai Uwe Broulik requested to merge work/kbroulik/noti-strip-trailing into master

Normally you wouldn't see this because a copy job has a destUrl (which is why I didn't notice) but when removing the filename to "go up" we need to strip the trailing slash, too.


File /foo/bar/baz:

  • Before: "Copying to /foo/bar/"
  • After: "Copying to /foo/bar"

Merge request reports

Loading