starred/CIPP

Fork 0

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

[GH-ISSUE #3923] [Bug]: Add to group option adds the wrong user resulting in TAP Error #1758

New issue

Closed

opened 2026-03-02 13:46:51 +03:00 by kerem · 3 comments

kerem commented

2026-03-02 13:46:51 +03:00

Owner

Originally created by @TSMNAdmin on GitHub (Apr 3, 2025).
Original GitHub issue: https://github.com/KelvinTegelaar/CIPP/issues/3923

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

When trying to create a TAP for an end user, we get the error "System.Collections.Hashtable". I've dug into this quite a bit, and what is happening is when we add a user to their tenants TAP Group using the ellipses menu, the wrong user this being added to the TAP group. It appears that the user that gets added using this method is the first user in the tenant alphabetically.

The workaround right now is to add the user to the TAP group through "Identity Management > Groups", which then successfully creates the TAP for the user, but this is inconvenient for our helpdesk staff.

Steps to reproduce the issue:

Navigate to Identity Management > Users
Find the user and the ellipses on the right side of the screen
Select "Add to group"
Select the TAP Group from the dropdown
Click "Confirm"
Navigate to Identity Management > Users
Find the user and the ellipses on the right side of the screen
Select "Create Temporary Access Password"
Click "Confirm"

Environment Type

Front End Version

v7.4.2

Back End Version

v7.4.2

Relevant Logs / Stack Trace

Originally created by @TSMNAdmin on GitHub (Apr 3, 2025). Original GitHub issue: https://github.com/KelvinTegelaar/CIPP/issues/3923 ### 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 When trying to create a TAP for an end user, we get the error "System.Collections.Hashtable". I've dug into this quite a bit, and what is happening is when we add a user to their tenants TAP Group using the ellipses menu, the wrong user this being added to the TAP group. It appears that the user that gets added using this method is the first user in the tenant alphabetically. The workaround right now is to add the user to the TAP group through "Identity Management > Groups", which then successfully creates the TAP for the user, but this is inconvenient for our helpdesk staff. Steps to reproduce the issue: 1. Navigate to Identity Management > Users 2. Find the user and the ellipses on the right side of the screen 3. Select "Add to group" 4. Select the TAP Group from the dropdown 5. Click "Confirm" 6. Navigate to Identity Management > Users 7. Find the user and the ellipses on the right side of the screen 8. Select "Create Temporary Access Password" 9. Click "Confirm" ### Environment Type Sponsored (paying) user ### Front End Version v7.4.2 ### Back End Version v7.4.2 ### Relevant Logs / Stack Trace ```plaintext ```

kerem

2026-03-02 13:46:51 +03:00

closed this issue
added the
bug

no-activity

unconfirmed-by-user
labels

kerem commented

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

Author

Owner

@github-actions[bot] commented on GitHub (Apr 3, 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 (Apr 3, 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:46:53 +03:00

Author

Owner

@github-actions[bot] commented on GitHub (Apr 13, 2025):

This issue is stale because it has been open 10 days with no activity. We will close this issue soon. If you want this feature implemented you can contribute it. See: https://docs.cipp.app/dev-documentation/contributing-to-the-code . Please notify the team if you are working on this yourself.

@github-actions[bot] commented on GitHub (Apr 13, 2025): This issue is stale because it has been open 10 days with no activity. We will close this issue soon. If you want this feature implemented you can contribute it. See: https://docs.cipp.app/dev-documentation/contributing-to-the-code . Please notify the team if you are working on this yourself.

kerem commented

2026-03-02 13:46:53 +03:00

Author

Owner

@KelvinTegelaar commented on GitHub (Apr 13, 2025):

Cant simulate this, might be more of a support issue than bug. Please contact support and we'll reopen is required :)

@KelvinTegelaar commented on GitHub (Apr 13, 2025): Cant simulate this, might be more of a support issue than bug. Please contact support and we'll reopen is required :)

kerem referenced this issue

2026-03-02 13:58:10 +03:00

[PR #1758] [MERGED] Webhook Alerts #3401

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#1758

No description provided.

Rows
Columns