Fabian Neugart
Fabian Neugart
Hey @argzdev , thanks for the investigation! I think we have seen what you have described. The token refresh only happens nearing the expiry date time.
Any chance this is getting fixed soon?
> Is it possible that your application in some other thread at the same time may be changing System.Properties in any way ? @harshitrjpt : Theoretically no, but honestly Android...
Would also vouch for a copy ```java System.getProperties().entrySet().stream().toArray()... // or Set.copyOf(properties.entrySet()) ``` That should make iterating safe but ofc we wouldn't know about any changes to the list. Don't know...
Anything missing here, @jack-berg ? Would be happy to see this in a release soon :).
Thanks for getting back (and with that detail)! I'll invest some time to get this reproduced on the test app there (`opentelemetry-android`), though it won't be a proper junit test...
Hey @jack-berg, was able to reproduce the crash with [this](https://github.com/open-telemetry/opentelemetry-android/commit/eecba047d3508fed4d59020d78fcd42fa96f7f6b) change. The fix with `clone` runs at ~10k ns. @laurit 's fix with [synchronizing](https://github.com/open-telemetry/opentelemetry-java/pull/6746#discussion_r1810052616) also works fine :1st_place_medal: . Let...
Didn't had much time recently, but looks good! Thanks for taking care of that :).
Same issue for me, @comphilip adding `oauth:` in the block helped! `prefect` is at `3.6.5`.