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

Fix redirect loop when certain checks are disabled, see #1209 #1219

Closed

Conversation

uwekamper
Copy link
Contributor

@uwekamper uwekamper commented Jan 2, 2024

This PR fixes the issue #1209 for domain and mail checks.

@stitch
Copy link
Collaborator

stitch commented Jan 19, 2024

Works as described, great pr!

image
image

stitch added a commit that referenced this pull request Jan 19, 2024
@stitch stitch mentioned this pull request Jan 19, 2024
stitch added a commit that referenced this pull request Jan 19, 2024
* resultscurrent() does not fail anymore when some checks are disabled

* make call to create_report more explicit

* list all checks even if some are disabled

* fix the same problem for mail

* fix lint for #1219

---------

Co-authored-by: Uwe Kamper <[email protected]>
Co-authored-by: Uwe Kamper <[email protected]>
@stitch stitch closed this Jan 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants