starred/CIPP

Fork 0

mirror of https://github.com/KelvinTegelaar/CIPP.git synced 2026-04-25 08:16:01 +03:00

[GH-ISSUE #3688] [Bug]: Password Pusher Integration - Generating incorrect links #1634

New issue

Closed

opened 2026-03-02 13:45:47 +03:00 by kerem · 4 comments

kerem commented

2026-03-02 13:45:47 +03:00

Owner

Originally created by @365techCIPP on GitHub (Feb 24, 2025).
Original GitHub issue: https://github.com/KelvinTegelaar/CIPP/issues/3688

Required confirmations before submitting

I can reproduce this issue on the latest released versions of both CIPP and CIPP-API.
I have searched existing issues (both open and closed) to avoid duplicates.
I am not requesting general support; this is an actual bug report.

Issue Description

Password Pusher integration is configured with our Pro Password Pusher account. PWPush URL: https://pwp.365tech.ca
Clicking on the Test but created a Password Push link but there is an extra /r at the end of the generated link. For example a link that is generated via the test button is:
https://pwp.365tech.ca/p/8h3cdst4ygib/r/r

The correct link should be the following instead:
https://pwp.365tech.ca/p/8h3cdst4ygib/r

We can confirm that the extra /r at the end of the link is also generated when a new user is created or when a user password is reset.

Environment Type

Front End Version

v7.2.3

Back End Version

v7.2.3

Relevant Logs / Stack Trace

Originally created by @365techCIPP on GitHub (Feb 24, 2025). Original GitHub issue: https://github.com/KelvinTegelaar/CIPP/issues/3688 ### Required confirmations before submitting - [x] **I can reproduce this issue on the latest released versions** of both CIPP and CIPP-API. - [x] **I have searched existing issues** (both open and closed) to avoid duplicates. - [x] I am **not** requesting general support; this is an actual bug report. ### Issue Description Password Pusher integration is configured with our Pro Password Pusher account. PWPush URL: https://pwp.365tech.ca Clicking on the Test but created a Password Push link but there is an extra /r at the end of the generated link. For example a link that is generated via the test button is: https://pwp.365tech.ca/p/8h3cdst4ygib/r/r The correct link should be the following instead: https://pwp.365tech.ca/p/8h3cdst4ygib/r We can confirm that the extra /r at the end of the link is also generated when a new user is created or when a user password is reset. ![Image](https://github.com/user-attachments/assets/ab7474d3-0110-4052-9e5f-b309d194bfa1) ![Image](https://github.com/user-attachments/assets/19603f38-0e28-4c14-864f-c2e1516b6c50) ### Environment Type Sponsored (paying) user ### Front End Version v7.2.3 ### Back End Version v7.2.3 ### Relevant Logs / Stack Trace ```plaintext ```

kerem

2026-03-02 13:45:47 +03:00

closed this issue
added the
bug

unconfirmed-by-user
labels

kerem commented

2026-03-02 13:45:48 +03:00

Author

Owner

@github-actions[bot] commented on GitHub (Feb 24, 2025):

Thank you for reporting a potential bug. If you would like to work on this bug, please comment:

I would like to work on this please!

Thank you for helping us maintain the project!

@github-actions[bot] commented on GitHub (Feb 24, 2025): Thank you for reporting a potential bug. If you would like to work on this bug, please comment: > I would like to work on this please! Thank you for helping us maintain the project!

kerem commented

2026-03-02 13:45:48 +03:00

Author

Owner

@JohnDuprey commented on GitHub (Feb 24, 2025):

hey @365techCIPP if you've enforced the password retrieval in policy on PWP side, ensure that you do not also do so in CIPP for the time being

@JohnDuprey commented on GitHub (Feb 24, 2025): hey @365techCIPP if you've enforced the password retrieval in policy on PWP side, ensure that you do not also do so in CIPP for the time being

kerem commented

2026-03-02 13:45:49 +03:00

Author

Owner

@365techCIPP commented on GitHub (Feb 24, 2025):

Thanks @JohnDuprey

We do enforce the Click to retrieve policy on the PWP side, but even after disabling the Click to retrieve password button on CIPP side it is still generating the extra /r on the end of the link.
We have disabled integration and reenabled it cleared browser cache, still the same.
Example:

https://pwp.365tech.ca/p/fe49cj_cellsyytji1e/r/r

@365techCIPP commented on GitHub (Feb 24, 2025): Thanks @JohnDuprey We do enforce the Click to retrieve policy on the PWP side, but even after disabling the Click to retrieve password button on CIPP side it is still generating the extra /r on the end of the link. We have disabled integration and reenabled it cleared browser cache, still the same. Example: ![Image](https://github.com/user-attachments/assets/3671d39a-0db2-43f3-93d9-2377dd70ceca) https://pwp.365tech.ca/p/fe49cj_cellsyytji1e/r/r

kerem commented

2026-03-02 13:45:49 +03:00

Author

Owner

@365techCIPP commented on GitHub (Feb 25, 2025):

For anyone else that faces the same issue the root cause of the issue is on the Password Pusher policy for Pushes.
1-Click Retrieval -> Default Value = true and set Force the default value to false

The Force the default value = true results in the double /r/r in the links generated in CIPP

@365techCIPP commented on GitHub (Feb 25, 2025): For anyone else that faces the same issue the root cause of the issue is on the Password Pusher policy for Pushes. 1-Click Retrieval -> Default Value = true and set Force the default value to **false** The Force the default value = true results in the double /r/r in the links generated in CIPP

kerem referenced this issue

2026-03-02 13:57:52 +03:00

[PR #1634] [CLOSED] Initial portal centralization #3332

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

starred/CIPP#1634

No description provided.

Rows
Columns