Skip to content

show banner with actual time range and limit if VL-Applied-* response headers significantly differ from UI values#1577

Open
AndrewChubatiuk wants to merge 1 commit into
masterfrom
vmui-modify-requested-time-and-data-range
Open

show banner with actual time range and limit if VL-Applied-* response headers significantly differ from UI values#1577
AndrewChubatiuk wants to merge 1 commit into
masterfrom
vmui-modify-requested-time-and-data-range

Conversation

@AndrewChubatiuk

@AndrewChubatiuk AndrewChubatiuk commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

if original start, end and limit query arguments were modified by proxy user may be confused by response. added support of VL-Applied-* headers to vmui which shows a alert with actual timerange and limit if returned by backend values significantly differ from set in UI

image

@AndrewChubatiuk
AndrewChubatiuk requested review from Loori-R and func25 July 7, 2026 06:28

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

3 issues found across 2 files

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

Comment thread app/vmui/packages/vmui/src/pages/QueryPage/hooks/useFetchLogs.ts Outdated
Comment thread app/vmui/packages/vmui/src/pages/QueryPage/hooks/useFetchLogs.ts Outdated
Comment thread app/vmui/packages/vmui/src/pages/QueryPage/hooks/useFetchLogs.ts Outdated
…* response headers significantly differ from UI values
@AndrewChubatiuk
AndrewChubatiuk force-pushed the vmui-modify-requested-time-and-data-range branch from 1ec33bb to 3aab600 Compare July 7, 2026 08:41
@AndrewChubatiuk AndrewChubatiuk changed the title app/vmui: sync limit and time range from VL-Applied-* response headers show banner with actual time range and limit if VL-Applied-* response headers significantly differ from UI values Jul 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant