Skip to content

Some macros can not be resolved in action URLs #1111

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

Jump to bottom
Closed
bobapple opened this issue Dec 17, 2024 · 1 comment · Fixed by #1118
Closed

Some macros can not be resolved in action URLs #1111

bobapple opened this issue Dec 17, 2024 · 1 comment · Fixed by #1118
Assignees
@sukhwinder33445
Labels
bug Something isn't working ref/IP
Milestone
1.1.4

Comments

@bobapple
Copy link
Member

Describe the bug

Using $host.state.last_state_change$ macro for action URLs breaks the detail view of affected hosts with the following error message:

Icinga\Module\Icingadb\Web\Navigation\Action::resolveMacro(): Return value must be of type string, DateTime returned

To Reproduce

  1. Creat a new Navigation Item of the type "Host Action" or "Service Action"
  2. Use the macro $host.state.last_state_change$ in the URL field
  3. Open detail view of affected Host or Service

Your Environment

  • Icinga Web version: 2.12.1
  • Icinga DB Web version: 1.1.3

Additional context

ref/IP/56978
@yhabteab
Copy link
Member

This is an Icing DB Web thing :)!

@yhabteab yhabteab transferred this issue from Icinga/icingadb Dec 17, 2024
@nilmerg nilmerg added this to the 1.2.0 milestone Dec 17, 2024
@nilmerg nilmerg added the bug Something isn't working label Dec 17, 2024
@sukhwinder33445 sukhwinder33445 self-assigned this Jan 9, 2025
@sukhwinder33445 sukhwinder33445 mentioned this issue Jan 9, 2025
Merged
@nilmerg nilmerg modified the milestones: 1.2.0, 1.1.4 May 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sukhwinder33445 sukhwinder33445

Labels
bug Something isn't working ref/IP
Projects
None yet
Milestone
1.1.4
Development

Successfully merging a pull request may close this issue.

Macros: Add missing support for DateTime type macro Icinga/icingadb-web
4 participants
@bobapple @nilmerg @sukhwinder33445 @yhabteab