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

Tenant allow/block list - removing entries not working #3567

Open
isgq-github01 opened this issue Feb 4, 2025 · 3 comments
Open

Tenant allow/block list - removing entries not working #3567

isgq-github01 opened this issue Feb 4, 2025 · 3 comments
Labels
bug Something isn't working unconfirmed-by-user

Comments

@isgq-github01
Copy link
Contributor

Description

Under Tenant Allow/Block Lists, selecting an entry > clicking the actions menu option > clicking Remove and then clicking Confirm - nothing happens after hitting Confirm. We do get this confirmation screen

Image

but the entry is still present. When testing the Remove-TenantAllowBlockListItems command, I did notice that I had to add -ListSubType Tenant as a parameter in order for the PowerShell command to work, whereas that's not present in the Invoke-RemoveTenantAllowBlockList.ps1 script.

Environment data

Sponsored, self-hosted
7.1.3 for back end and front end version.
@isgq-github01 isgq-github01 added bug Something isn't working unconfirmed-by-user labels Feb 4, 2025
Copy link

github-actions bot commented Feb 4, 2025

Thank you for creating a bug. Please make sure your bug is indeed a unique case by checking current and past issues, and reading the complete documentation at https://docs.cipp.app/
If your bug is a known documentation issue, it will be closed without notice by a contributor. To confirm that this is not a bug found in the documentation, please copy and paste the following comment: "I confirm that I have checked the documentation thoroughly and believe this to be an actual bug."

Without confirming, your report will be closed in 24 hours. If you'd like this bug to be assigned to you, please comment "I would like to work on this please!".

@isgq-github01
Copy link
Contributor Author

I confirm that I have checked the documentation thoroughly and believe this to be an actual bug.

@OfficialEsco
Copy link
Contributor

The (main) issue here is actually that in v6 the actions were commented out since Remove-TenantAllowBlockListItems requires the ListType (FileHash, Url, Sender, BulkSender, Recipient, IP) which Get-TenantAllowBlockListItems DOES NOT RETURN... and we're doing a bulk request which runs trough all of the ListTypes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working unconfirmed-by-user
Projects
None yet
Development

No branches or pull requests

2 participants