bfreuden

@pendula95 I don't get how we can solve that "magically" by adding some synchronization in vertx-auth. Doing so would require to change Vert.x documentation. Users of vertx-auth would be required...

@pmlopes This bug is nasty. I agree the reproducer is pushing Vert.x very hard with the delays but, believe me, in a real world scenario I get multiple errors per...

@pmlopes Thank you very much for your answers. I'll run my reproducer with the latest 4.3.0-SNAPSHOT (probably this evening) and I will let you know. I've tried using the modified...

Hi @pmlopes, I still have ConcurrentModificationException errors that seem to be gone with this modification: ``` public class AuthorizationsImpl implements Authorizations { @Override public Authorizations add(String providerId, Set authorizations) {...

I'm realizing only the `getOrCreateAuthorizations` has to be modified. Sorry (panic mode...)

I've given vert.x 4.3.0-SNAPSHOT a try (fresh clones of vert.x vertx-web and vertx-auth, master branch) and I do get ConcurrentModificationException as well, in addition to 403 errors: ``` java.util.ConcurrentModificationException at...

I forgot to say that I modified some javadoc comments, hoping that they will be clearer. I can revert to the original comments if you don't like the new ones.

Wait... the PR is containing a line that should not be here: ```java user.authorizations().clear(providerId); ``` That is linked with the issue described here: https://groups.google.com/g/vertx/c/vTCLEQsv1N0 I will remove it for the...

@vietj @bboyz269 I am ready to discuss the topic with you if you are willing to. I'm under the impression it is not so complex to implement but, because the...

Yes. Having a look at the methods of the `io.vertx.ext.mongo.MongoClient` interface, I'm realizing it could almost be called `io.vertx.ext.mongo.MongoDatabase` (even `io.vertx.ext.mongo.MongoCollection`). Let me show you the `MongoClient.java` file without its...