ramccor / esmska

Automatically exported from code.google.com/p/esmska
0 stars 0 forks source link

nova sms brana - smska.cz #290

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
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 stanisla...@gmail.com on 29 Jul 2010 at 12:38

Attachments:

GoogleCodeExporter commented 9 years ago
Zatim k videni na "docasene" adrese smska.glux.eu

Original comment by stanisla...@gmail.com on 29 Jul 2010 at 12:39

GoogleCodeExporter commented 9 years ago
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 comment by kamil.paral on 31 Jul 2010 at 10:16

GoogleCodeExporter commented 9 years ago
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 comment by kamil.paral on 2 Aug 2010 at 7:38

GoogleCodeExporter commented 9 years ago
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 comment by stanisla...@gmail.com on 3 Aug 2010 at 11:17

GoogleCodeExporter commented 9 years ago
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 comment by kamil.paral on 5 Aug 2010 at 6:34

GoogleCodeExporter commented 9 years ago
A samozřejmě jsem zapomněl přiložit tu bránu.

Original comment by kamil.paral on 5 Aug 2010 at 6:34

Attachments:

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

Original comment by kamil.paral on 10 Aug 2010 at 5:14

GoogleCodeExporter commented 9 years ago
Bump. Jak to vypadá?

Original comment by kamil.paral on 19 Aug 2010 at 8:24

GoogleCodeExporter commented 9 years ago
Behem pondeli se na to podivam a doladim ten zbytek - byl jsem na dovoleny mimo 
dosah internetu.

Original comment by stanisla...@gmail.com on 21 Aug 2010 at 8:15

GoogleCodeExporter commented 9 years ago
Nějaká změna?

Original comment by kamil.paral on 23 Sep 2010 at 8:15

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

Original comment by kamil.paral on 2 Oct 2010 at 9:13

GoogleCodeExporter commented 9 years ago
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 comment by miloslav...@hosting90.cz on 22 Feb 2011 at 10:08

Attachments:

GoogleCodeExporter commented 9 years ago
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 comment by kamil.paral on 23 Feb 2011 at 9:11

Attachments:

GoogleCodeExporter commented 9 years ago
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 comment by miloslav...@hosting90.cz on 28 Feb 2011 at 8:27

Attachments:

GoogleCodeExporter commented 9 years ago
This issue was closed by revision 01f17da259.

Original comment by kamil.paral on 28 Feb 2011 at 5:42

GoogleCodeExporter commented 9 years ago
Díky, bude součástí další verze programu.

Original comment by kamil.paral on 28 Feb 2011 at 5:42