Skip to content

Update Checkport references#2103

Merged
theCalcaholic merged 1 commit intonextcloud:develfrom
n0rs3:fix/checkport_references
Apr 14, 2026
Merged

Update Checkport references#2103
theCalcaholic merged 1 commit intonextcloud:develfrom
n0rs3:fix/checkport_references

Conversation

@n0rs3
Copy link
Copy Markdown
Collaborator

@n0rs3 n0rs3 commented Apr 4, 2026

Updated the references to checkport in the is_port_open comments.


Hello @theCalcaholic, it's been a while since we last spoke, how are you? 😄 (I changed my username on GitHub from ZendaiOwl to n0rs3)

I'm making this pull request to update the references in the comments to point to where I have the repository now.

Updated the references to checkport in the is_port_open comments.

Signed-off-by: Victor-ray, S <git@zendai.net.eu.org>
@theCalcaholic
Copy link
Copy Markdown
Collaborator

theCalcaholic commented Apr 14, 2026

Hey @n0rs3 , nice to hear from you :)
Thank you for the update.
I'm not too comfortable with hosting the port check somewhere where it may cause you expenses, though.
I'll move it to my own server soon, if you don't mind.
I'd set it up so it automatically pulls your docker image once per day, so you could still easily push updates.

@theCalcaholic theCalcaholic merged commit a5db307 into nextcloud:devel Apr 14, 2026
6 of 8 checks passed
@n0rs3
Copy link
Copy Markdown
Collaborator Author

n0rs3 commented Apr 14, 2026

If you rather want to host it yourself that's okay with me, I also have no issue to keep running it :)

It's not gotten close to hitting the limits in the past 2 years it's been running so I'm not particularly worried. The limit is over 430 simultaneous request per seconds for an entire month after all. The text is mostly to discourage frivolous use outside the admin panel.


How are things with you otherwise?

I finally graduated in march :D

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.

2 participants