Skip to content
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.

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

nightly2 - Change to HTTP(s) monitor Request Timeout does not persist #5337

Closed
1 task done
matthuisman opened this issue Nov 12, 2024 · 1 comment Β· Fixed by #5352
Closed
1 task done

nightly2 - Change to HTTP(s) monitor Request Timeout does not persist #5337

matthuisman opened this issue Nov 12, 2024 · 1 comment Β· Fixed by #5352
Labels
area:dashboard The main dashboard page where monitors' status are shown bug Something isn't working help wanted May need your help to test or answer releaseblocker blocking bugs encountered with a new release
Milestone

Comments

@matthuisman
Copy link

πŸ“‘ I have found these related issues/pull requests

none

πŸ›‘οΈ Security Policy

Description

No response

πŸ‘Ÿ Reproduction steps

Create http(s) monitor to https://www.example.com
Set the Request Timeout to 5
Save
Edit the monitor
Notice Request Timeout is back to its original value (cant recall - 46 seconds or something)

πŸ‘€ Expected behavior

The monitor saves the new value for Request Timeout and uses it

πŸ˜“ Actual Behavior

Monitor does not save the request timeout value

🐻 Uptime-Kuma Version

nightly2

πŸ’» Operating System and Arch

Docker

🌐 Browser

Chrome

πŸ–₯️ Deployment Environment

  • Runtime: docker
  • Database: builtin mariadb
  • Filesystem used to store the database on: docker mounted volume
  • number of monitors: 1

πŸ“ Relevant log output

No response

@matthuisman matthuisman added the bug Something isn't working label Nov 12, 2024
@CommanderStorm CommanderStorm added this to the 2.0.0-beta.1 milestone Nov 12, 2024
@CommanderStorm CommanderStorm added help wanted May need your help to test or answer releaseblocker blocking bugs encountered with a new release area:dashboard The main dashboard page where monitors' status are shown labels Nov 12, 2024
@louislam
Copy link
Owner

Confirmed. That is strange, I can't remember which pull request touched this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:dashboard The main dashboard page where monitors' status are shown bug Something isn't working help wanted May need your help to test or answer releaseblocker blocking bugs encountered with a new release
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants