search

PaulWoitaschek / websms-connector-developergarden

WebSMS Developergarden Connector
2 stars 2 forks source link

Überlange SMS Fehlfunktion #4

Closed neniuideo closed 10 years ago

neniuideo commented 10 years ago

Mit Version 0.2 Beta wird eine SMS mit 153 Zeichen mit dem falschen Absender gesendet. Anstatt +49123456789 ist der Absender +4949123456789

Scheint aber nen Fehler im Telekom Server zu sein.

mySMS, keine Ahnung warum die Längenbeschränkung nicht greift. Wenns ein Fehler in mySMS ist kann ich den ja dort mal später melden.

Ankommen tun auch nur 129 Zeichen, bei deutlich mehr als 255 kommt ein Fehler vom Server (zu lang).

Vermutlich wäre es sinnvoll im Connector den String auf 129 Zeichen zu beschneiden? Oder bei mehr als 129 ne Excerption zu werfen?

BTW: SMS mit Sonderzeichen werden vom Connector gar nicht angenommen? mySMS wechselt bei solchen automatisch zum nächsten.

cu

neniuideo commented 10 years ago

Kann es sein das setLimitLength für die max. Absenderlänge ist? ;)

http://www.androidadb.com/source/felixb-websms-723dff4/connectors/gmx/src/de/ub0r/android/websms/connector/gmx/ConnectorGMX.java.html

cu

claell commented 10 years ago

Meinst du WebSMS?

neniuideo commented 10 years ago

Nein, ich nutze https://play.google.com/store/apps/details?id=com.mysms.android.sms&hl=de

Aber du hast meinen zweiten Kommentar gelesen? setLimitLength wird nicht das tun was du erwartest. Jedenfalls sieht das für mich so aus.

BTW: Ich habe mal den innosend free Connector probiert. Der hat das selbe Problem, der Anbieter hängt Werbung an so das die 160 Zeichen nicht nutzbar sind. Eine 160 Zeichen SMS wird vom innosend Server angenommen aber nie weitergeleitet (weil zu lang). Ich werde dann mal nen Bugreport beim innosend Connector machen (der Anbieter gibt alle 50 Minuten eine SMS, wird also noch dauern ehe ich alles probiert habe ;) ).

Weniger als 160 Zeichen erlaubt scheint also generell ein Problem bei den Web SMS Connectoren zu sein.

cu

neniuideo commented 10 years ago

Habe mal hier nen Bug gepostet: https://github.com/felixb/websms-connector-innosend/issues/1 Genau das selbe Problem.

cu

claell commented 10 years ago

Könnte gut sein, dass mit setLimitLength die maximale Länge o.ä. gesetzt werden kann, im Beispiel ist das 10.

Ich wusste nicht, dass mySMS auch die Connectoren von WebSMS nutzen kann.

PaulWoitaschek commented 10 years ago

Bitte mal hier neue ausprobieren und vorher die alte runterschmeißen ;-) https://github.com/Ph1b/de.ph1b.dgard/blob/master/apkfiles/dgard.0.3.1-beta.apk

Die Limitlength ist schon für den Text. Zwar zeigt er beim Schreiben jeweils von 160 an aber sobald man mehr als 129 getippt hat, löscht WebSMS das direkt wieder. Zusätzlich hab ich noch ne Exception reingeschmissen, sodass er falls irgendwie die 129 durchkommen, das Senden mit einer Fehlermeldung abbricht.

claell commented 10 years ago

Kann man die 160, die angezeigt werden auch ändern?

PaulWoitaschek commented 10 years ago

Hab ich auch schon nach gesucht. Ich glaube bei dem Connector mit dem alten Auth ging das auch, aber keine Ahnung wie. Meiner Meinung nach wäre es ja am sinnvollsten, wenn WebSMS das direkt anzeigt; es macht ja keinen Sinn, dem User zu sagen: Du hast noch 20 Zeichen und ihm dann zu sagen, dass das nächste schon zu viel ist.

claell commented 10 years ago

Ich meine auch, dass der alte Connector das konnte. Kannst du ja mal in der Dokumentation für Connectoren nachsehen, falls es sowas gibt.

neniuideo commented 10 years ago

Cool, du hast irgendwas anderst gemacht als letztes mal ;) Bei mehr als 129 Zeichen deaktiviert mySMS den Connector und springt automatisch zum nächsten (das ging letztes mal nicht). Also ich finde das so klasse. Genauso springt mySMS zum nächsten Unicode tauglichen wenn ich nen Unicode Zeichen verwende.

BTW: Deine Längenberechnung https://github.com/Ph1b/de.ph1b.dgard/blob/master/DevGardenConnector/src/main/java/de/ph1b/dgard/DevCon.java#L162 wird so nicht funktionieren. Das "€" benötigt z.B. zwei Zeichen. http://de.wikipedia.org/wiki/SMS-Kodierung Ich glaube dir Connector Tools bringen ne Längenberechnung mit.

cu

PS FYI: Wegen neuinstalieren... "Use a custom sender" nicht wählen geht nicht. Dort gabs nen Fehler wegen nicht registrierten Absender.

PaulWoitaschek commented 10 years ago

Probier mal die 0.3.2 im apkordner. Wenn ich das use custom sender weglasse, kann ich senden, ohne eine Nummer registriert zu haben. Geht das bei noch wem nicht?

Das use default benutze ich nicht zur Auswahl, weil Leute sonst nicht den Text bei der Checkbox lesen und merken, dass man Nummern wirklich erst registrieren muss und nicht einfach irgendwelche nehmen kann.

neniuideo commented 10 years ago

Das use default benutze ich nicht zur Auswahl, weil Leute sonst nicht den Text bei der Checkbox lesen und merken, dass man Nummern wirklich erst registrieren muss und nicht einfach irgendwelche nehmen kann.

Ist nen Argument ;)

Ist ja im Prinzip auch egal. Man wird seine Nmmer bei DevGarden aktivieren und dann den Connector einmal einstellen. Und dann muss man da nie wieder (es sei denn man wechselt seine Nummer) ran.

cu