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

Find the problem #741

Closed
qq1176914912 opened this issue May 6, 2024 · 18 comments
Closed

Find the problem #741

qq1176914912 opened this issue May 6, 2024 · 18 comments
Assignees
Labels
bug Something isn't working

Comments

@qq1176914912
Copy link

1、When you create a user group, select Delete after the user group is created.
See video1
2、After you add a child member to a group, the child member is displayed on the left. When you remove the child member, the child member on the left still exists.
See video2
3、After adding sub-members to the group, directly click the group in the left large menu again, the group page will keep turning in a circle, and no response will be clicked.
See video3
video:
videos.zip

@qq1176914912 qq1176914912 changed the title Problems with Groups Find the problem May 6, 2024
@qq1176914912
Copy link
Author

qq1176914912 commented May 6, 2024

I have downloaded your latest project, the database uses POSTGRE, there is a problem when operating the user's "Add claim", when I add a random data, click "save", 5001 has an error:
image
image
This issue did not exist before the update.

@simpleidserver simpleidserver self-assigned this May 7, 2024
@simpleidserver simpleidserver added the bug Something isn't working label May 7, 2024
@simpleidserver simpleidserver moved this to In Progress in Release 5.0.0 May 7, 2024
thabart added a commit that referenced this issue May 7, 2024
Ticket #741 : Fix issues in the Group screen + add missing migration scripts
@simpleidserver
Copy link
Owner

Hello,

Both issues are fixed in the master branch :)

@simpleidserver simpleidserver moved this from In Progress to Done in Release 5.0.0 May 7, 2024
@qq1176914912 qq1176914912 reopened this May 8, 2024
@qq1176914912
Copy link
Author

I am sorry to ask you, after I started the project with the original library, I found that the 5001 project would keep appearing this "warn" message, I tried to regenerate the library, but there was no such problem, so will this problem affect the project?
image

@qq1176914912 qq1176914912 reopened this May 8, 2024
@qq1176914912
Copy link
Author

User sessions that have expired are still active, and clicking the "Revoke sessions" button at the top will not reset the status of those expired sessions:
image
The reason may be that there is old data (expired sessions) in the database before, but the state in the database is 0.

video4.mp4

thabart added a commit that referenced this issue May 8, 2024
@simpleidserver
Copy link
Owner

Hello,

I have fixed a minor issue in the master branch.
Previously, a serialization exception was being thrown when a user session was revoked. Could you please try again?

KR,

SID.

@qq1176914912
Copy link
Author

Hello,

I have fixed a minor issue in the master branch. Previously, a serialization exception was being thrown when a user session was revoked. Could you please try again?

KR,

SID.

Thank you. Problem solved

@qq1176914912
Copy link
Author

qq1176914912 commented May 9, 2024

When the database uses POSTGRE and uses the new library, let the project automatically create tables and start your SCIM project, the user information exists in the Scim project:
image
Click on "Launch" and then click the button circled below again
image
5001 The following error occurs:
image
image
This error only appears in the first place. If I click "Launch" again and then click the button "Import users and groups", there will be no problem and it will show that the import is successful:
image
However, the user table does not contain the user information in the scim
image

Ldap is not sure if this problem also exists

@simpleidserver
Copy link
Owner

The problem has been resolved in the 'master' branch. An exception was thrown because the importation algorithm attempted to insert a record into the 'GroupUser' table, but the 'GroupId' value was empty.

Could you please remove all records from the 'ExtractedRepresentations' and 'ExtractedRepresentationsStaging' tables and try again? I have created ticket '#745' to provide the option to retry failed commands in the 'Masstransit' message bus.

@qq1176914912
Copy link
Author

The problem has been resolved in the 'master' branch. An exception was thrown because the importation algorithm attempted to insert a record into the 'GroupUser' table, but the 'GroupId' value was empty.

Could you please remove all records from the 'ExtractedRepresentations' and 'ExtractedRepresentationsStaging' tables and try again? I have created ticket '#745' to provide the option to retry failed commands in the 'Masstransit' message bus.

Thank you. Problem solved

@qq1176914912
Copy link
Author

There is no unique judgment for adding a role position on the client. If you add the same role, an error will be reported when selecting a role for the group:
video5.zip

@qq1176914912
Copy link
Author

Hello, I found that when "Certificate Authorities" added "Certificate store", the error message was incomplete, resulting in the page card owner:
video6.zip

@qq1176914912
Copy link
Author

Hello, I cannot detect the Certificate using "Certificate store". What is the problem?
image
image

thabart added a commit that referenced this issue May 10, 2024
@simpleidserver
Copy link
Owner

I made the following modifications in the master branch:

  • Display an error message when the user tries to insert a duplicate client's role.
  • Display an error message when the "CryptographicException" is thrown.
  • The certificate cannot be imported by the thumbprint because the certificate is invalid. The error message has been updated to be more accurate.

@qq1176914912
Copy link
Author

Hello, I have a question about Auditing. I created a new library and I need to access and log in. Why is there a word "failure" in Auditing? I did not fail during the login process, and why do "Client Authentication Success" and "Token Issued Success" appear twice?
image

@simpleidserver simpleidserver moved this from Done to In Progress in Release 5.0.0 May 11, 2024
@simpleidserver
Copy link
Owner

There was a small error in the AuthorizationController; the event AuthorizationFailureEvent was published in the wrong place.
Now, this event is published only when something goes wrong in the AuthorizationController.

The message "Token issued success" can appear twice if the website uses the refresh token to get a new access token.
The auditing messages have been updated to offer more visibility.

@simpleidserver simpleidserver moved this from In Progress to Done in Release 5.0.0 May 13, 2024
@qq1176914912
Copy link
Author

Hello, excuse me.
If you use pwd+webauthn in ACRS, an error occurs when you enter the account password:
https://github.com/simpleidserver/SimpleIdServer/assets/79817742/e70c69da-176b-4ddd-b5f7-62b4557ff2eb
It does not seem to be the reason why the account is not bound to webauthn, because I bind webauthn to the account, the problem will still occur.

@qq1176914912 qq1176914912 reopened this May 17, 2024
@qq1176914912
Copy link
Author

When there are steps, if you do not enter the account name and password, click login directly, the error message will cause the above step hours (only displayed on the hour, the actual did not disappear).

video8.mp4

@simpleidserver
Copy link
Owner

Both issues are fixed in the master branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
No open projects
Status: Done
Development

No branches or pull requests

2 participants