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

Aplikacja niepotrzebnie wysyła nadmiarowe informacje #85

Closed
jasisz opened this issue Apr 8, 2020 · 4 comments
Closed

Aplikacja niepotrzebnie wysyła nadmiarowe informacje #85

jasisz opened this issue Apr 8, 2020 · 4 comments

Comments

@jasisz
Copy link
Contributor

jasisz commented Apr 8, 2020

Następujące pola nie są potrzebne przy wysyłaniu zapytania get_status:

  • platform
  • os_version
  • device_type
  • app_version
  • lang

Ich wysyłanie (i zapisywanie) nie spełnia w systemie żadnej roli i jest zbędne.

@jakublipinski
Copy link
Contributor

Bluetooth stack, jego możliwości i ograniczenia jest mocno zależny od urządzenia na którym działa i wersji systemu operacyjnego. Wiedząc które urządzania są wykorzystywane przez użytkowników, możemy priorytetyzować poprawki i testy.
lang używane jest do zwracania użytkownikowi z serwera komunikatów w jego języku. Może też nam pozwolić na zdecydowanie które nowe języki uwzględnić w tłumaczeniach (np. ukraiński?)

@jasisz
Copy link
Contributor Author

jasisz commented Apr 10, 2020

@jakublipinski Jeżeli takie są cele, to myślę że dla pewności i przejrzystości warto byłoby:

  • wskazać je w specs
  • nie wiązać tych danych z konkretnym użytkownikiem

Co uważasz?

@D0han
Copy link
Contributor

D0han commented Apr 10, 2020

Tłumaczenie na różne języki powinno odbywać się po stronie aplikacji klienckiej, nie po stronie serwera. Serwer powinien zwracać jedynie statusy zdefiniowane w protokole.

@KoderFPV
Copy link
Contributor

KoderFPV commented Jun 5, 2020

Wątek mocno nie aktualny.
Na tą chwile zamykam w ramach sprzątania starych wątków.
Jeżeli sądzicie że aplikacja nadal wysyła niepotrzebne dane dajcie nam znać :)

@KoderFPV KoderFPV closed this as completed Jun 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants