Git Product home page Git Product logo

Comments (15)

thomasaugsten avatar thomasaugsten commented on July 16, 2024 2

Is already done. It should not occur further notes. There was a direct change. I will have a look if we a default value in the repo. Yes it was adapted.

from cwa-app-android.

mlenkeit avatar mlenkeit commented on July 16, 2024 1

@Ein-Tim thanks for raising this! This message is designed as a security means to protect the server in case there are too many key submissions. The number (i.e. 10) is configurable, so that we can adjust it if there was too much traffic. At the moment this mechanism should is not activated.

As you correctly mentioned, CWA anyway only considers the last 10 days and this message should not show up. My assumption is that by some means, the app submitted keys that were older than 10 days. I'll try to figure out why this happens. Is this the only report so far that you've seen?

For what it's worth, it's not an error message but just a notice 😉

from cwa-app-android.

mlenkeit avatar mlenkeit commented on July 16, 2024 1

@mlenkeit FYI: https://twitter.com/Jimmyfreak88/status/1617591246427672576?s=20 seems like Ursache10 also plays a role here.

@Ein-Tim thanks for referencing the tweet. This might indeed be a series of events. Ursache10 has always been very hard (not to say impossible) to reproduce, let's see if we find a link between the two.

from cwa-app-android.

dsarkar avatar dsarkar commented on July 16, 2024 1

@Ein-Tim This issue is currently being analysed, and I have actually seen it also on iOS. I will also forward your additional information to the internal ticket. Many thanks.

from cwa-app-android.

Ein-Tim avatar Ein-Tim commented on July 16, 2024

@mlenkeit Sounds good! Thank you and let me know if you need any more details!

from cwa-app-android.

Ein-Tim avatar Ein-Tim commented on July 16, 2024

@mlenkeit FYI: https://twitter.com/Jimmyfreak88/status/1617591246427672576?s=20 seems like Ursache10 also plays a role here.

from cwa-app-android.

Ein-Tim avatar Ein-Tim commented on July 16, 2024

@mlenkeit

Have you tried warning yourself yet? Here's a second user reporting the same issue: https://twitter.com/muc_neko/status/1617893129507729408?s=61&t=zkhUfqeLvv1PZzwKxfMKyA

Maybe the Android implementation is faulty? I issued a SRS warning on my iPhone 6s (I deleted all contacts via the OS settings previously) and did not get an error like this.

from cwa-app-android.

Ein-Tim avatar Ein-Tim commented on July 16, 2024

@mlenkeit / @dsarkar / @thomasaugsten

I'd like to escalate this issue. From reports on Twitter I tend to say that this is not an isolated issue but always occurs. I will try warning myself on an Android device with version 3.0.1 installed and error logging active. Let's see if I'll get the same error.

First users on Twitter are starting to interpret this warning/informational pop-up as a confirmation that the SRS feature is broadly used to troll others: https://twitter.com/trollvanknall/status/1618762048246251520?s=46&t=or1WPnZNeDFj-OmlLAXdpQ

from cwa-app-android.

Ein-Tim avatar Ein-Tim commented on July 16, 2024

@dsarkar In this case this issue can be transferred to the cwa-documentation repository.

from cwa-app-android.

thomasaugsten avatar thomasaugsten commented on July 16, 2024

We adopt adapted the configuration to prevent this warning which maybe confuse the user

from cwa-app-android.

Ein-Tim avatar Ein-Tim commented on July 16, 2024

@thomasaugsten

Okay so this was already done on server side? Is there a corresponding PR?

from cwa-app-android.

MikeMcC399 avatar MikeMcC399 commented on July 16, 2024

@thomasaugsten

We adopt the configuration to prevent this warning which maybe confuse the user

Did you mean "adapt" not "adopt"?
If it is "adopt" what is the configuration you are adopting?

from cwa-app-android.

mlenkeit avatar mlenkeit commented on July 16, 2024

It's an environment variable configuration setting that is not reflected in any of the repositories. We changed the value of this variable, which was set to 10:
https://github.com/corona-warn-app/cwa-server/blob/50063b0df5e16eee6a137e64ae35307492347f26/services/submission/src/main/resources/application.yaml#L23-L24

from cwa-app-android.

Ein-Tim avatar Ein-Tim commented on July 16, 2024

@mlenkeit

Means the change isn't visible in the repo?

from cwa-app-android.

Ein-Tim avatar Ein-Tim commented on July 16, 2024

Congrats, this error made it into ZDF Wiso: https://www.zdf.de/verbraucher/wiso/wiso-vom-30-januar-2023-100.html (at 01:26).

But seriously, as this should be fixed now, I'll close this issue.

from cwa-app-android.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.