Skip to content
This repository was archived by the owner on Nov 8, 2021. It is now read-only.

nova sms brana - smska.cz #312

Closed
kparal opened this issue May 23, 2015 · 16 comments
Closed

nova sms brana - smska.cz #312

kparal opened this issue May 23, 2015 · 16 comments

Comments

@kparal
Copy link
Owner

kparal commented May 23, 2015

V rubehu tohoto tydne bude na smska.cz spustena nova SMS brana. Prikladam gateway script
pro jeji API.

Original issue reported on code.google.com by stanislav.petr on 2010-07-29 12:38:24


- _Attachment: [[CZ]SMSKA.cz.gateway](https://storage.googleapis.com/google-code-attachments/esmska/issue-290/comment-0/[CZ]SMSKA.cz.gateway)_ - _Attachment: [CZ]SMSKA.cz.png
![[CZ]SMSKA.cz.png](https://storage.googleapis.com/google-code-attachments/esmska/issue-290/comment-0/[CZ]SMSKA.cz.png)_
@kparal
Copy link
Owner Author

kparal commented May 23, 2015

Zatim k videni na "docasene" adrese smska.glux.eu

Original issue reported on code.google.com by stanislav.petr on 2010-07-29 12:39:23

@kparal
Copy link
Owner Author

kparal commented May 23, 2015

Díky, budu se snažit na to kouknout přes víkend. Jinak na těch stránkách je překlep
v "přez".

Original issue reported on code.google.com by kamil.paral on 2010-07-31 10:16:43

  • Labels added: Type-Operator
  • Labels removed: Type-Defect

@kparal
Copy link
Owner Author

kparal commented May 23, 2015

Název brány jsem dal do lowercasu.

A mé dotazy:

1. Má to umožňovat posílat zprávy pouze do ČR a nikam jinam? V tom případě dáme
getSupportedPrefixes jako ["+420"].

2. Umí to posílat na libovolné české číslo? V tom případě dáme getPreferredPrefixes
jako [].

3. Brána opravdu přijme zprávu až o 1530 znacích? Jak to koreluje s tím, že při odesílání
je natvrdo nastaveno 'max_length' na '5'?

4. "Neplatná autorizace" - ten paznak je úmysl?

5. Nekontroluješ tam dost věcí. Například když není číslo příjemce podporováno, tak
to uživateli vůbec neřekneš.

Jinak doporučil bych odesílat POSTem a ideálně přes https, pokud to lze.

Original issue reported on code.google.com by kamil.paral on 2010-08-02 19:38:36

@kparal
Copy link
Owner Author

kparal commented May 23, 2015

1. Posila to zpravy zatim jen do CR. Dalsi operatori budou postupne nasmlouvany.
2. Neni povoleno odesilani na premova cisla (9xxxx). Zatim neni povoleno posilat SMS
na pevne linky. Povolene cile jsou Telefonica O2 GSM, Telefonica O2 NMT, Vodafone,
T-Mobile GSM, U:Fon
3. Tam bude asi chyba. Marametr max_length rika do kolika zprav muze byt zprva rozdelena.
Pokud je API predan telsi text je zkracen na delku 160*max_length. Jedna se o ochranu
v API proti vycerpani kreditu pri chybnem predani dat. Max. hodnota v max_length je
9, ale doporucuji posilat SMS rozdelene max. do peti zprav. Plnou delku dle specifikace
GSM sice umime odeslat, ale vetsina mobilnich telefonu to uz neumi spravne prijmout
a spolehlive prijimaji multisms do peti casti.
4. Odpoved je v XML, takze ten "paznak" je XML entita nahrazujici znak dlouhe a. Do
budoucna budou pridany chybove kody, kterych se pak pujde chytit lepe.
5. Doplnim, ale predpokladal jsem ze o to se postara getSupportedPrefixes

Original issue reported on code.google.com by stanislav.petr on 2010-08-03 11:17:28

@kparal
Copy link
Owner Author

kparal commented May 23, 2015

1,2. Ok, tak to uděláme tak jak navrhuju, a těch pár výjimek (prémiová čísla, pevná
linka) ošetři kontrolou návratového kódu požadavku, přijde mi to tak nejjednodušší
(a nemusíš měnit skript při případných změnách v bráně).

3. Takže getMaxChars() měním na 800 (5*160).

4. A já myslel, že posíláš XML v UTF-8, takže není nutné zaměňovat české znaky za XML
entity. No, to byl jen takový dotaz, je to v podstatě jedno (dokud to bude fungovat).

5. Viz první a druhý bod, doporučoval bych doplnit.

Aktuální verzi skriptu přikládám. Na řádek 104 jsem přidal takovou obecnou hlášku "neznámá
chyba", pokud nějaká chyba nastane, ale není odchycena žádnou z předchozích kontrol.
Je to ale jen takové nouzové řešení. Pokud v těch XML opravdu posíláš srozumitelné
chybové hlášky, doporučoval bych ti prostě vyparsovat tu hlášku z XML a zobrazit ji
přímo uživateli (viz EXEC.ERROR_GATEWAY_MESSAGE a skripty jiných bran). Jen ty XML
entity to budou možná trochu komplikovat.

Jinak se mi to zdá dobrý (samozřejmě nemohu vyzkoušet funkčnost, ale to si snad ověříš,
včetně hraničních stavů).

Pošli upravenou verzi brány a myslím že to můžeme poslat uživatelům.

Original issue reported on code.google.com by kamil.paral on 2010-08-05 18:34:13

@kparal
Copy link
Owner Author

kparal commented May 23, 2015

A samozřejmě jsem zapomněl přiložit tu bránu.

Original issue reported on code.google.com by kamil.paral on 2010-08-05 18:34:54


- _Attachment: [[CZ]smska.cz.gateway](https://storage.googleapis.com/google-code-attachments/esmska/issue-290/comment-6/[CZ]smska.cz.gateway)_

@kparal
Copy link
Owner Author

kparal commented May 23, 2015

Tak jak? Brána je kousek od zařazení, stačí upravit pár řádků.

Original issue reported on code.google.com by kamil.paral on 2010-08-10 17:14:56

@kparal
Copy link
Owner Author

kparal commented May 23, 2015

Bump. Jak to vypadá?

Original issue reported on code.google.com by kamil.paral on 2010-08-19 20:24:03

@kparal
Copy link
Owner Author

kparal commented May 23, 2015

Behem pondeli se na to podivam a doladim ten zbytek - byl jsem na dovoleny mimo dosah
internetu.

Original issue reported on code.google.com by stanislav.petr on 2010-08-21 08:15:34

@kparal
Copy link
Owner Author

kparal commented May 23, 2015

Nějaká změna?

Original issue reported on code.google.com by kamil.paral on 2010-09-23 20:15:32

@kparal
Copy link
Owner Author

kparal commented May 23, 2015

Jakmile budeš mít zájem do zařadit, tak dej vědět. Do té doby uzavírám.

Original issue reported on code.google.com by kamil.paral on 2010-10-02 09:13:56

@kparal
Copy link
Owner Author

kparal commented May 23, 2015

Ahoj,
mám tu verzi s nějakejma opravama. kdyžtak na to prosím mrkni a komentáře mi prosím
napiš na mail (pač se může stát, že jinak na to zapomenu)

Original issue reported on code.google.com by miloslav.semler@hosting90.cz on 2011-02-22 10:08:42


- _Attachment: [[CZ]smska.cz.gateway](https://storage.googleapis.com/google-code-attachments/esmska/issue-290/comment-12/[CZ]smska.cz.gateway)_

@kparal
Copy link
Owner Author

kparal commented May 23, 2015

Email by ti měl přijít automaticky tím, že to zde okomentuji. V podstatě je brána hotova
(udělal jsem pár kosmetických úprav), jen dva komentáře:
1. Správce brány mám nechat na Stanislav Petr, nebo to mám změnit na tebe?
2. Posílat uživatelské heslo přes HTTP GET je strašná prasárna. Uživatelům se to bude
ukládat do logu a když mi ho sem poté pošlou kvůli nějakému problému, tak si každý
bude moci přečíst jejich přihlašovací údaje. To ti lidé pak z vaší brány asi nebudou
moc nadšeni. Přepnout posílání na HTTP POST je triviální změna, opravdu bych to doporučoval
změnit.

Original issue reported on code.google.com by kamil.paral on 2011-02-23 21:11:47


- _Attachment: [[CZ]smska.cz.gateway](https://storage.googleapis.com/google-code-attachments/esmska/issue-290/comment-13/[CZ]smska.cz.gateway)_

@kparal
Copy link
Owner Author

kparal commented May 23, 2015

tak úpravy zapracovány. UID/heslo se posílá přes post. Co se týče správce, je to asi
jedno. Tady to vždycky někdo převezme.

Original issue reported on code.google.com by miloslav.semler@hosting90.cz on 2011-02-28 08:27:24


- _Attachment: [[CZ]SMSKA.cz.gateway](https://storage.googleapis.com/google-code-attachments/esmska/issue-290/comment-14/[CZ]SMSKA.cz.gateway)_

@kparal
Copy link
Owner Author

kparal commented May 23, 2015

This issue was closed by revision 01f17da259.

Original issue reported on code.google.com by kamil.paral on 2011-02-28 17:42:18

@kparal
Copy link
Owner Author

kparal commented May 23, 2015

Díky, bude součástí další verze programu.

Original issue reported on code.google.com by kamil.paral on 2011-02-28 17:42:48

@kparal kparal closed this as completed May 23, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant