Skip to content

Commit

Permalink
fix: logfiles endpoint missing authentication
Browse files Browse the repository at this point in the history
The skServer/logfiles was not protected by admin authentication
as a result of an earlier server API paths reorganisation. This
fix adds authentication to the correct path so that log files
are now protected.
  • Loading branch information
tkurki committed Jun 25, 2024
1 parent e937e58 commit 67ab790
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/interfaces/logfiles.js
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ module.exports = function (app) {
}

function mountApi(app) {
app.securityStrategy.addAdminMiddleware('/logfiles')
app.securityStrategy.addAdminMiddleware(`${SERVERROUTESPREFIX}/logfiles/`)
app.get(`${SERVERROUTESPREFIX}/logfiles/`, function (req, res) {
listLogFiles(app, (err, files) => {
if (err) {
Expand Down

0 comments on commit 67ab790

Please sign in to comment.