Problem/Motivation

Like other modules (Captcha, Flood Control, ...) this module could also integrate nicely and easily with https://www.drupal.org/project/crowdsec to send signals!

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Comments

anybody created an issue. See original summary.

zeeshan_khan’s picture

Thanks for the suggestion — this is a great idea and aligns well with the purpose of the module.

Integration with the CrowdSec module makes sense, especially for emitting signals on suspicious password reset activity (e.g., repeated reset attempts, invalid token usage, or potential enumeration patterns).

That said, this would likely be implemented as an optional integration to avoid adding a hard dependency. It would also require careful consideration around signal thresholds to avoid noise.

Happy to consider this as a feature — contributions or proposals for a minimal implementation are very welcome.

zeeshan_khan’s picture

Added support for Crowdsec in latest release 1.0.2

zeeshan_khan’s picture

Status: Active » Needs review
anybody’s picture

This is the commit from @zeeshan_khan
https://git.drupalcode.org/project/secure_password_reset_log/-/commit/f0...

LGTM.

@zeeshan_khan maybe you should ping the Crowdsec maintainer @jurgenhaas to let him know!

zeeshan_khan’s picture

Sure thanks @Julian

anybody’s picture

Thank you @zeeshan_khan! "Julian" - see my profile https://www.drupal.org/u/anybody ;)

zeeshan_khan’s picture

Yeah saw that already sorry I didn't notice your name before :)

jurgenhaas’s picture

Status: Needs review » Reviewed & tested by the community

This is nice, thanks for providing that integration. If you wanted to add CrowdSec to the ecosystems of this module on the project page of it, then it would automatically show up on the integration list.

zeeshan_khan’s picture

Status: Reviewed & tested by the community » Fixed

Thank you @jurgenhaas, really appreciate you taking the time to review it!

Would it be possible to get secure_password_reset_log added to the CrowdSec project's ecosystem listing? That should help site builders discover the integration more easily.

Marking as fixed since the integration shipped in 1.0.2.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

jurgenhaas’s picture

Would it be possible to get secure_password_reset_log added to the CrowdSec project's ecosystem listing? That should help site builders discover the integration more easily.

That's what I meant by my comment in #9 that you can get there yourself by adding crowdsec into the ecosystem of this module. You find that when you edit the project page of your module.

zeeshan_khan’s picture

Ah, got it — thank you for clarifying! Added to the project page now.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.