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

Uściślenie protokołu sieciowego #56

Closed
metiulekm opened this issue Mar 3, 2020 · 3 comments
Closed

Uściślenie protokołu sieciowego #56

metiulekm opened this issue Mar 3, 2020 · 3 comments
Labels
help wanted Extra attention is needed
Milestone

Comments

@metiulekm
Copy link
Contributor

metiulekm commented Mar 3, 2020

W specyfikacji o protokole jest napisane bardzo niewiele:

Komunikacja odbywa się za pośrednictwem protokołu TCP.

Wiadomości wysyłane są w formacie JSON.

Jednak taka definicja protokołu pozostawia bardzo dużo do życzenia; nic nie wiadomo o tym, jak konkretnie należy enkodować wiadomości JSON w strumieniu bajtów TCP.

@BartoszChrostowski
Copy link
Collaborator

Jesteśmy otwarci co do propozycji dotyczące protokołu, nie zgłębialiśmy tego tematu.

UTF-8

@BartoszChrostowski BartoszChrostowski added the help wanted Extra attention is needed label Mar 3, 2020
@BartoszChrostowski BartoszChrostowski pinned this issue Mar 3, 2020
@metiulekm
Copy link
Contributor Author

UTF-8

Nie o to mi chodziło (ale moja wina, zapomniałem że słówko "encoding" ma konkretne znaczenie). Ale jeśli już o tym mowa, to IMO troszkę overkill, ASCII nie wystarczy?

Jesteśmy otwarci co do propozycji dotyczące protokołu, nie zgłębialiśmy tego tematu.

U nas chyba wyglądało to tak, że po prostu na strumieniu TCP wysyłaliśmy kolejne wiadomości oddzielone znakami nowej linii (przy czym każda OFC musiała się zmieścić w jednej linii). Jest to chyba mniej lub bardziej standardowy zapis strumienia wiadomości JSON. Trzeba by to było jakoś uściślić jeszcze, warto by było wprowadzić limit długości jednej wiadomości oraz rozważyć ponowne łączenie się (w szczególności, co się stanie jeśli ktoś spróbuje się połączyć, a połączenie tego ID już istnieje?).

Temat bezpieczeństwa zupełnie olewamy? (Nie obrażę się, jest to chyba troszkę out of scope).

@BartoszChrostowski
Copy link
Collaborator

UTF-8. Nie ma żadnej zalety użycia ASCII

U nas chyba wyglądało to tak, że po prostu na strumieniu TCP wysyłaliśmy kolejne wiadomości oddzielone znakami nowej linii (przy czym każda OFC musiała się zmieścić w jednej linii). Jest to chyba mniej lub bardziej standardowy zapis strumienia wiadomości JSON. Trzeba by to było jakoś uściślić jeszcze, warto by było

To jest duplikacja #55

Nie wiem jeszcze jak z ponownym łączeniem. Wolałbym kończyć działanie na zerwanie jakiegokolwiek, ale jeśli są jakieś przeciwskazania, których nie widzę, trzeba będzie dodać taką możliwość.

Temat bezpieczeństwa definitywnie olewamy, nie ma sesnu skoro to nie jest wymagane.

@BartoszChrostowski BartoszChrostowski added this to the v1.1 milestone Mar 6, 2020
BartoszChrostowski added a commit that referenced this issue Mar 8, 2020
Close #55
Close #56
Close #57
BartoszChrostowski added a commit that referenced this issue Mar 8, 2020
Close #55
Close #56
Close #57
@BartoszChrostowski BartoszChrostowski unpinned this issue Mar 13, 2020
BartoszChrostowski added a commit that referenced this issue Mar 16, 2020
* Fix response priorities

* Add timers specification

Close #58

* Specify communication

Close #55
Close #56
Close #57

* Fix movement

Close #66

* Clarify information exchange rules

Co-authored-by: Jakub Drak Sbahi <[email protected]>
Co-authored-by: BartoszChrostowski <[email protected]>
Co-authored-by: BartoszChrostowski <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants