Skip to content

Monitoring Logs: read a log file as text/plain instead of failing content negotiation #6970

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
delchev merged 1 commit into from
Aug 27, 2026
+5 −2
Merged

Monitoring Logs: read a log file as text/plain instead of failing content negotiation #6970

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 4 additions & 1 deletion ...tion-core/src/main/resources/META-INF/dirigible/application-core/shell/js/services/api.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -96,7 +96,10 @@ App.services.api = {
// X-Requested-With marks the call as programmatic for browsers without Sec-Fetch-Mode: the
// server then answers an expired session with a PLAIN 401 (no Basic challenge), so the
// browser's native login dialog never pops over a background poll.
const headers = { 'Accept': 'application/json', 'X-Requested-With': 'XMLHttpRequest' };
// A caller may pin the Accept header for THIS call ({ accept: 'text/plain' }) - e.g. the
// monitoring Logs page, whose file endpoint produces text/plain and answers 406 to a JSON-only
// Accept before the handler even runs. JSON stays the default.
const headers = { 'Accept': opts.accept || 'application/json', 'X-Requested-With': 'XMLHttpRequest' };
if (!isForm) headers['Content-Type'] = 'application/json';
// A caller may pin the request language for THIS call ({ language: 'bg' }) — e.g. Print, where the
// chosen print language must drive the multilingual data overlay, not the UI locale. Absent the
Expand Down
2 changes: 1 addition & 1 deletion .../resources-monitoring/src/main/resources/META-INF/dirigible/monitoring/js/services/ops.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -105,7 +105,7 @@ window.MonitoringOps = (() => {
logFiles: () => get('/services/ide/logs/'),

/** One log file, whole. It is never polled - the page tails what it loaded. */
logFile: (file) => get('/services/ide/logs/' + encodeURIComponent(file)),
logFile: (file) => App.services.api.get('/services/ide/logs/' + encodeURIComponent(file), { ...ABSOLUTE, accept: 'text/plain' }),

/**
* Run a read that must not take the page down with it. A monitoring screen exists to show what is
Expand Down
Loading