protego-safe / ios Goto Github PK

Is your feature request related to a problem? Please describe.
I wanted to add a one letter nickname and the app didn't allow me to do so.

Describe the solution you'd like
One letter nicknames should be allowed.

Describe alternatives you've considered
Well, I can always add a longer nickname as I think it is basically meaningless but the limitation doesn't make any sense from the user's perspective.

Aplikacja odpytuje serwer i pokazuje Użytkownikowi pomarańczowy status

Po pierwsze chciałbym pochwalić wybór licencji GPL!
iOS jest o tyle problematyczny, że jeśli chcecie akceptować kod z zewnątrz (a zakładam że to jest jeden z celów publikacji) to każdy autor musi się zgodzić na publikację w AppStore. Najlepiej od razu wymagać CLA z odpowiednim zapisem o prawach autorskich, inaczej w przyszłości ktoś może odebrać Wam prawa do swojego kodu za złamanie licencji GPL (ograniczenie ilości instalacji narzucane przez AppStore).

Info: https://apple.stackexchange.com/a/59495
Oczywiście nie jestem prawnikiem.

TBD

Na bieżąco kasujemy informacje o napotkanych urządzeniach starsze niż 2 tyg

Aktualnie struktury BeaconId oraz ExpiringBeaconId są używane zarówno w roli lokalnej (własna pula BeaconId) oraz zdalnej (BeaconId pobierane od innych użytkowników). Jedną z propozycji usprawnienia implementacji jest wprowadzenie nowych typów LocalBeaconId, RemoteBeaconId lub odpowiedników.

Aplikacja odpytuje serwer i pokazuje Użytkownikowi jeden z trzech statusów:

• czerwony
• pomarańczowy
• zielony

Aktualnie tworząc body do requestów GCP ignorujemy błędy przez uzycie try?

Trzeba dodać obsługę błędów z ucyiem zwykłego try

Originally posted by @mateuszmackowiak in #60

Po uruchomieniu, aplikacja sprawdza czy zainstalowana wersja jest najnowsza i w przeciwnym wypadku Użytkownik proszony jest o zainstalowanie nowszej wersji i przekierowany jest do odpowiedniego sklepu z aplikacjami.

There were no major changes in 4.2.2 just minor fixes but still we need to deliver them into the CHANGELOG file.
ETA closest release.

Wskazówki zawarte w readme pomijają kwestie podpisywania aplikacji certyfikatem, który obsługuje notyfikacje push, przez co próba kompilacji kończy się błędem:

"Your development team, [nazwa], does not support the Push Notifications capability."

Proszę o dodanie self-signed certyfikatów do repozytorium, tak jak robi to np. Telegram, które umożliwią kompilacje każdemu zainteresowanemu.

Inną zaletą tych certyfikatów będzie umożliwienie zewnętrzynm programistom kompilacji projektu i tworzenia pull requestów. W Niemczech stworzony został specjalny profil dla społeczności (ENACommunity), który to umożliwia, wraz ze zmockowanym Exposure Notification API.

Stan obecny:

• Aktualnie moduł rozgłaszający jest włączony cały czas.

Stan docelowy:

• Moduł rozgłaszający powinien zostać włączony dopiero w momencie, gdy dostępne będą Beacon IDs do rozgłaszania.

Describe the bug
I received the notification that I was exposured... i click on the notification, it goes to the app but no more text or history of notyfications is available.
So if accidentaly I will clear my notyfications I will not know that I am „in danger”.

Expected behavior
A new tab with notyfications history, to be able to see them and read them.
Screenshots
The notyfications was available here:

Smartphone (please complete the following information):

• Device model: iPhone SE -2nd gen
• OS:1 iOS 14.0.1
  Additional context
  I think that this user (M4g3r) has the same issue
  

wg. projektu: ProteGO-Safe/specs#10

TBD

Aktualna implementacja przeprowadza proces synchronizacji poprzez prostą maszynę stanów. Stan SynchronizedBeaconId jest aktualnie nieużywany w praktyce i może warto go usunąć z implementacji.

Ze względu na to, że numer telefonu nie będzie obowiązkowy (ProteGO-Safe/specs#34 (comment)) użytkownicy mogą zarejestrować się bez podania nr telefonu. API zostanie ustalone tutaj: ProteGO-Safe/backend#67

iOS14 Aplikacja przestała działać po wtorkowym update, (brak połączenia z Apple coronavirus API, czerwony przycisk włącz powiadomienia o koronawirusie przestał działać, ale też przy pytaniu o zwykłe powiadomienia przy pierwszym uruchomieniu nie dało się wcisnąć dalej)
Wydaje mi się że w poniedziałek już na iOS 14 jeszcze mi działało

The CoronaRiskScoring project is trying to assess how different contact tracing apps are being parametrized. For this purpose, we need to know how many TEKs get published per day, and their risk levels (all public information).

Project considered
We are looking at ProteGO-Safe in our issue #10.

What we need
We need to have a procedure describing how to pull down the keys reliably.

What we see
We see the documentation here.

This looks relevant:

• EN_STORAGE_URL = https://exp.safesafe.app/
• endpoints :/index.txt, /{filename}

However when we reach out to endpoint https://exp.safesafe.app/index.txt or https://exp.safesafe.app/1 we get that the "specified key does not exist".

It does not look like we would need to authenticate though. Is the directory currently empty?

Aplikacja działa w tle i na bieżąco zapisuje informacje o wykrytych urządzeniach. W przypadku jeśli aplikacja zostanie wyłączona podejmuje próbę włączenia się ponownie.

Describe the bug
My exposure log was not checked since almost a week.

Expected behavior
I thought that my exposure log will be checked every 24h.

Screenshots

Smartphone (please complete the following information):

• Device: iPhone XS
• OS: iOS 13.5.1
• Version 4.2.0
• my device has internet connection and it wasn’t in the battery saving mode

Zapraszam do przeczytania ogłoszenia

Opis prozypozycji rozwiązania

Podczas review PR związanego z UI ekranami pokazywanymi podczas wysyłania historii spotkań padł pomysł (#57 (comment)) aby uwspólnić nawigację tej częsci z resztą aplikacji.

Aktualnie wykorzystywany jest na UINavigationController, a reszta aplikacji korzysta z customowego rozwiązania podmieniającego child view controllery
Ze wzgledu na to, że jest to sprawa czysto architektoniczna i nie ma wpływu dla użytkownika, postanowiliśmy przesunąć pracę nad tą częścią po wypuszczeniu wersji 1.0

Aby rozwiązanie było użyteczne aplikacja powinna nadawać oraz odbierać Beacon ID w tle według możliwości. Należy mieć na uwadze zużycie baterii oraz ograniczenia nakładane na planer systemu operacyjnego.

Stan aktualny:

• Aplikacja zawsze rozgłasza się, gdy jest aktywna.
• Gdy aplikacja przechodzi do tła, rozgłaszanie jest włączane i wyłączane naprzemiennie (bez tego mechanizmu iOS bardzo szybko usuwał proces).
• Używane są BackgroundTask na czas połączenia oraz rozgłaszania aby kontynuuować aktywność po wejściu do tła.
• Używany jest BackgroundProcessingTask do wybudzania aplikacji co ok. 15 min.

Wątek ten otwieram w celu dalszej dyskusji co do usprawnień w tym obszarze. Wszelkie uwagi będą mile widziane.

J.w.
Skoro Ministerstwo Cyfryzacji wydało aplikacje, to musi mieć dostęp do chociaż jednego konta deweloperskiego Apple. Moim zdaniem to jest bardzo dobry pomysł żeby devteam zainstalował chociaż na jednym urządzeniu tą betę i to załatał zanim Apple zacznie wydawać te aktualizacje i aplikacja nie będzie działać u obywateli.

Describe the bug
Aplikacja nie działa na iOS 12.5.1 mimo ze Apple wprowadził do niej exposure notification.

To Reproduce
Steps to reproduce the behavior:
N/A
Current behavior
Aplikacja wyswietla notyfikacje ze znajduje sie w trybie offline i trzeba uruchomic moduł BT, (moduł BT jest uruchomiony) a aplikacja pozostaje w trybie offline
Expected behavior
Aplikacja będzie współpracowała z iOS 12.5.1.
Screenshots
N/A
Smartphone (please complete the following information):

• Device model: iPhone 6
• OS: iOS 12.5.1
  Additional context
  N/A

Obecna dokumentacja nie informuje o tym jak zweryfikować czy kod źródłowy, który jest w tym repozytorium został faktycznie użyty do skompilowania aplikacji, która jest w AppStore.

Proszę o stworzenie dokumentacji, która umożliwi to szerszej publiczności. Przykład takiej dokumentacji: https://core.telegram.org/reproducible-builds#reproducible-builds-for-ios

Opis prozypozycji rozwiązania

Projekt, nad którym pracuje kilka osób w tym samym czasie powoduje wiele problemów podczas merge kilku równoległych PR. Konflikty najczęściej dotyczą plików projektowych, w tym przypadku .xcworkspace

Opis alternatywnych rozwiązań

Proponuję użycie jednego z narzędzi:

• https://github.com/yonaskolb/XcodeGen
• https://github.com/tuist/tuist

Dodatkowe informacje

Jeśli uważacie to za dobre rozwiązanie, mogę się tym zająć

Ze względów bezpieczeństwa proponuję zmianę flagi dostępności .always w pliku SecretsGeneratorSpecs na .afterFirstUnlockThisDeviceOnly.

Btw. Dostępność .always jest już oznaczana przez Apple jako deprecated - https://developer.apple.com/documentation/security/ksecattraccessiblealways

W dniu dzisiejszym wykonałem świeżą instalację aplikacji na iOS 14.1 (wersja finalna) na iPhone XR.

Ustawienia powiadomień COVID w iOS wyczyszczone. Dane aplikacji ProteGO usunięte i aplikacja również. Następnie świeża instalacja aplikacji i w momencie włączenia funkcji powiadomień w systemie apka przechodzi do systemowych powiadomień o COVID (API iOS) i po przejściu do tego ekranu jest on pusty… Nie da się nic wybrać. Próbowałem w drugą stronę, czyli wybrać z systemu Polskę jako kraj powiadomień i iOS przekierowuje do Appstore i sugeruje instalację ProteGO.
Tym samym nie da się obecnie poprawnie dokończyć instalacji aplikacji i używania powiadomień i wymiany informacji via Bluetooth.

Screeny (na czerwono miejsca, w które klikam):

I na koniec:

Opis błędu

deviceType w requeście do API wysyła nazwę urządzenia nadaną przez użytkownika, zamiast typ urządzenia.

Kroki

Oczekiwany zachowanie aplikacji

deviceType powinien wysyłać typ urządzenia

Urządzenie (proszę wypełnij poniższe informacje dotyczące urządzenia):

• System operacyjny: iOS
• Wersja aplikacji ProteGO Dev 1.0.37

Środowisko

• Wersja środowiska: dev

Aktualna implementacja nie odczytuje i nie propaguje informacji o TX power. Jest ona konieczna wraz z wartością RSSI do lepszego oszacowania odległości między urządzeniami.

Sugerowane zmiany:

• Dodanie i propagowanie TX power pobranego z Advertisement Data
• Usprawnienie mechanizmu zbierania próbek RSSI dla lepszego oszacowania odległości:
  • Zwiększenie ilości pobieranych próbek podczas rozgłaszania i połączenia.
  • Ustalenie algorytmu, który decyduje, czy wartość RSSI jest "stabilna" i użyteczna do kalkulacji.

Stan aktualny:

• Brak pobierania informacji o TX power
• RSSI jest sczytywane podczas procesu discovery, ale opcja AllowDuplicates nie jest aktualnie używana (w czasie działania w tle nie jest ona dostępna).
• RSSI jest pobierane tylko raz podczas połączenia.
• W momenice, gdy inne urządzenie zapisuje swój BeaconId (klasa Advertiser) nie jest pobierane RSSI. (czy istnieje API po stronie PeripheralManager by to osiągnąć?)

This issue is addressed here.

Opis błędu

Po naciśnięciu przycisku menu na głównym ekranie aplikacja crashuje.

Kroki

1. Otwórz aplikację.
2. Przejdź onboarding bez podawania numeru telefonu.
3. Nacisnąć przycisk menu w prawym głównym rogu.
4. Aplikacja crashuje

Oczekiwany zachowanie aplikacji

Aplikacja nie powinna crashować i powinna pokazać menu.

Urządzenie (proszę wypełnij poniższe informacje dotyczące urządzenia):

• Nazwa: iPhone 6 Plus
• System operacyjny: iOS
• Wersja systemu operacyjnego: iOS 12.4.3
• Wersja aplikacji: 1.0.47

Środowisko

• Wersja środowiska: Dev

Dodatkowe informacje

W konsoli urządzenia widzę taką wiadomość: Fatal error: Use of unimplemented initializer 'init(nibName:bundle:)' for class 'ProteGO.HistoryRootViewController'. Na innym urządzeniu (iPhone XS) aplikacja otwiera menu prawidłowo.

Aktualnie API nie posiada opcji nasłuchiwania stanu Bluetooth, co byłoby przydatne dla użytkowników aplikacji.

W przypadku braku dostępu do Internetu, aplikacja informuje o tym i nie pozwala przejść dalej.

Opis błędu

Użytkownik może wpisać i potwierdzić nr telefonu o długości innej niż 9 cyfr

Kroki

1. Przejdź przez onboarding
2. Wpisz nr telefonu z zakresu <0,8> <9, )
3. Potwierdź nr telefonu

Oczekiwany zachowanie aplikacji

Aplikacja powinna uniemożliwić potwierdzenie nr telefonu o innej długości niż 9 cyfr.
Pole nr telefonu powinno zawierać komunikat błędnej walidacji numeru.

Urządzenie (proszę wypełnij poniższe informacje dotyczące urządzenia):

• Nazwa: iPhone 7
• System operacyjny: iOS
• Wersja systemu operacyjnego: iOS 13.4
• Wersja aplikacji 1.0.35

Środowisko

• Wersja środowiska: stg

Kolejne uruchomienie
Aplikacja sprawdza czy Bluetooth jest włączony. Jeśli nie, użytkownik proszony jest o włączenie Bluetooth. Aplikacja nie pozwala pójść dalej bez włączenia Bluetooth (chyba że nie da się skutecznie sprawdzić stanu Bluetooth, wtedy tylko przypomina i przechodzi).

Pierwsze uruchomienie
W przypadku kiedy aplikacja uruchamiana jest po raz pierwszy (nie mamy user_id), następuje rejestracja aplikacji. Użytkownikowi pokazywane są ekrany startowe:

Powitanie w aplikacji
Akceptacja regulaminu. Regulamin jako link otwierany w przeglądarce.
Wyjaśnienie jak działa aplikacja (może być kilka ekranów)
Prośba o włączenie Bluetooth. Aplikacja nie przechodzi dalej bez włączenia Bluetooth (chyba że nie da się skutecznie sprawdzić stanu Bluetooth, wtedy przechodzi).

Na jednym telefonie możemy mieć zainstalowane różne wersje aplikacji dev, stag, prod. Niech rozgłaszają się różnym uuid.

As there are many foreigners living in Poland it would be a good idea to have English localization available in the app. Even if most contents from Government won’t be translated the general app information and questionnaire should be possible to do in English.

Wymagania:
Działa na iOS i Android. Obie platformy widzą się nawzajem.
Maksymalnie oszczędza baterię.
Zapisujemy siłę sygnału
Działa w tle. Po zamknięciu aplikacja wybudza się automatycznie.
Co godzinę rozgłaszamy inny identyfikator.
Wykryte urządzenia możemy śledzić np. w BugFender

Prośba o podanie numeru telefonu. Na stałe umieszczony jest przedrostek +48.
Po potwierdzeniu, użytkownik otrzymuje wiadomość SMS na podany numer i wpisuje 6-cyfrowy kod z wiadomości.

Yesterday I've got notification about exposure to COVID, but app is giving me opposite information as seen on screenshot.

Smartphone:

• Device model: [iPhone 7]
• OS: [iOS 14.3]

Is your feature request related to a problem? Please describe.
I'm ashamed to be living in a country where a PWA app is priced at 2.5m PLN.
Describe the solution you'd like
Make an actual native app, or ask someone else to do this properly.

Describe alternatives you've considered
Not using this app.

Additional context
I beg you, create an actual iOS app that works as it should. I can even make one but please, remove this monstrosity out of the internet.

Aktualna implementacja skanera jest konfigurowana zestawem parametrów przedstawionych poniżej (ustawionych w czasie wstępnych testów). Wartości te powinny zostać odpowiednio zmodyfikowane przed wypuszczeniem aplikacji:

• BackgroundTaskEarliestBeginDate (15 min) - czas uruchomienia BackgroundProcessingTask w celu kontynuuowania skanowania/rozgłaszania. Według dokumentacji Apple jest to wartość zalecana.
• PeripheralSynchronizationCheckInSec (5 sec) - cykliczny czas sprawdzania stanu znanych urządzeń.
• PeripheralSynchronizationTimeoutInSec (15 sec) - czas po którym skaner zrywa połączenie, gdy peryferium nie jest w stanie wykonać wszystkich przewidzianych operacji.
• PeripheralIgnoredTimeoutInSec (60 sec) - czas ograniczający wysłanie kolejnego "spotkania" w przypadku, gdy jesteśmy w stanie zidentyfikować dane urządzenie jako takie same.
• PeripheralMaxConcurrentConnections (3) - maksymalna liczba równoległych połączeń obsługiwanych przez klasę CentralManager.
• PeripheralMaxConnectionRetries (3) - maksymalna liczba ponowień połączenia w przypadku, gdy przewidujemy, że urządzenie będzie miało "stały" MAC address (Android zmienia MAC address co połączenie).
• AdvertisingOnTimeout (15 sec) - czas przez który rozgłaszanie jest włączone w tle.
• AdvertisingOffTimeout (45 sec) - czas przez który urządzenie nie rozgłasza się w tle.
• ScanningOnTimeout (15 sec) - czas przez który skanowanie jest włączone w tle.
• ScanningOffTimeout (45 sec) - czas przez który urządzenie nie skanuje w tle.

Ze względu na to, że numer telefonu nie będzie obowiązkowy (ProteGO-Safe/specs#34 (comment)) użytkownicy będą proszeni o podanie jakiegoś tekstowego identyfikatora (TBD) przed wysłaniem historii spotkań (encounters). Identyfikator będzie wysyłany na serwer, wraz z historią i tam weryfikowany.