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

Discover Echo 2G Fix #1486

Closed
wants to merge 20 commits into from

Conversation

reloxx13
Copy link
Contributor

@reloxx13 reloxx13 commented Jan 1, 2018

-add meta service xml for echo plus 2g (maybe not neccessery, needs more testing)
-add switch between belkin for type1 devices and rootdevice for type2 devices
-st and usn response change depending on the switch

#1357, #1450

tested and working with echo dot 2g, echo 2g, echo plus 2g
waiting for test result for echo show 2g

Restart Echo Device once if nothing gets found (Alexa is blocking devices if the response was wrong in past).

pre-build firmware (all languages):
ALL-firmware-5.10.0g-fix-echo-type2-v4.zip

@reloxx13
Copy link
Contributor Author

reloxx13 commented Jan 1, 2018

22:53:52 m-search*http/1.1
host:239.255.255.250:1900
man:"ssdp:discover"
mx:5
st:urn:schemas-upnp-org:device:avm-aha:1

22:53:52 UPP: WeMo TYPE: rootdevice
22:53:52 UPP: WeMo Antwort gesendet zu 192.168.178.1:42100

not working with echo show 2g :S

the echo show seems to not send an udp request.
this is a log while discovering from echo show and their are only fritzbox discovering requests inside.

log.txt

arendst added a commit that referenced this pull request Jan 2, 2018
5.10.0h
 * Fix Wemo Emulation for Gen 2 devices (#1486)
@arendst
Copy link
Owner

arendst commented Jan 2, 2018

The missed echo show 2g may be caused by a too small UDP buffer.

On line 26 the UDP buffer is defined as UDP_BUFFER_SIZE 200

You might want to raise it to 300 to see if the request is in the remainder of the M-SEARCH packet.

EDIT: Ah I see. There is no request at all. Strange...

@reloxx13
Copy link
Contributor Author

reloxx13 commented Jan 2, 2018

Unfortunately i dont own the device myself, so i cannot sniff the network to check what the show is sending.
the sonos and fire tv stick with alexa remote are not working, too :S

ill give feedback if i get some more infos.

@jimjimminy
Copy link

FYI this bwssytems/ha-bridge#860 suggests a timing issue with the UPNP discovery as being a potential cause

@arendst
Copy link
Owner

arendst commented Jan 5, 2018

Working on the timing issue. About to release soon.

@jimjimminy
Copy link

Great news thanks, I tested the 5.10.0g-fix-echo-type2-v4 build last night and it wasn't discovered by my Echo 2. Hopefully the timing issue will resolve this. Happy to test this out if it helps.

arendst added a commit that referenced this pull request Jan 5, 2018
5.10.0i
 * Add device function pointers
 * Moved Command and JSON
English language defines to one file (i18n.h)
 * Update it-IT.h language
file (#1449)
 * Fix Wemo and Hue emulation by adding M-Search response
delay (#1486)
arendst added a commit that referenced this pull request Jan 7, 2018
5.11.0 20180107
 * Minor webpage HTML optimizations (#1358)
 * Updated
German translation (#1438)
 * Change Sonoff Pow Energy MQTT data message
and consolidate Status 8 into Status 10
 * Change ADS1115 default
voltage range from +/-2V to +/-6V (#1289)
 * Change text to Active for 3
minutes (#1364)
 * Change Wemo SetBinaryState to distinguish from
GetBinaryState (#1357)
 * Change output of HTTP command to valid JSON
and Array only (#1363)
 * Removed all MQTT, JSON and Command language
defines from locale files and set fixed to English (#1473)
 * Renamed
commands Color2,3,4 to Color3,4,5
 * Fix BME280 calculation (#1051)
 *
Fix Sonoff Bridge missed learned key if learned data contains 0x55 (End
of Transmission) flag (#1095, #1294)
 * Fix PWM initialization in
Dimmer/Color mode (#1321)
 * Fix Wemo Emulation (#1357)
 * Fix display
of build date and time in non-english locale (#1465)
 * Fix Wemo and Hue
emulation by adding M-Search response delay (#1486)
 * Add libraries
Adafruit_BME680-1.0.5, Adafruit_Sensor-1.0.2.02, TasmotaSerial-1.0.0 and
TSL2561-Arduino-Library
 * Add command Color2 to set color while keeping
same dimmer value
 * Add device function pointers
 * Add support for
SenseAir S8 CO2 sensor
 * Add color led signal to Carbon Dioxide (CO2)
sensors using defines CO2_LOW and CO2_HIGH in user_config.h
 * Add
support for Domoticz Air Quality sensor to be used by MH-Z19(B) and
SenseAir sensors
 * Add support for PZEM004T energy sensor
 * Add
support for iTead SI7021 temperature and humidity sensor by
consolidating DHT22 into AM2301 and using former DHT22 as SI7021 (#735)

* Add support for BME680 using adafruit libraries (#1212)
 * Add support
for MH-Z19(B) CO2 sensor (#561, #1248)
 * Add multipress support and
more user configurable GPIO to Sonoff Dual R2 (#1291)
 * Add support for
TSL2561 using adafruit library (#661, #1311)
 * Add support for SHT3x
(#1314)
 * Add support for Arilux LC06 (#1414)
 * Add Italian language
file (#1449)
 * Add 2nd Gen Alexa support to Wemo emulation discovery
(#1357, #1450)
 * Add define for additional number of WS2812 schemes
(#1463)
joecotton pushed a commit to joecotton/Sonoff-Tasmota that referenced this pull request Jan 8, 2018
5.11.0 20180107
 * Minor webpage HTML optimizations (arendst#1358)
 * Updated
German translation (arendst#1438)
 * Change Sonoff Pow Energy MQTT data message
and consolidate Status 8 into Status 10
 * Change ADS1115 default
voltage range from +/-2V to +/-6V (arendst#1289)
 * Change text to Active for 3
minutes (arendst#1364)
 * Change Wemo SetBinaryState to distinguish from
GetBinaryState (arendst#1357)
 * Change output of HTTP command to valid JSON
and Array only (arendst#1363)
 * Removed all MQTT, JSON and Command language
defines from locale files and set fixed to English (arendst#1473)
 * Renamed
commands Color2,3,4 to Color3,4,5
 * Fix BME280 calculation (arendst#1051)
 *
Fix Sonoff Bridge missed learned key if learned data contains 0x55 (End
of Transmission) flag (arendst#1095, arendst#1294)
 * Fix PWM initialization in
Dimmer/Color mode (arendst#1321)
 * Fix Wemo Emulation (arendst#1357)
 * Fix display
of build date and time in non-english locale (arendst#1465)
 * Fix Wemo and Hue
emulation by adding M-Search response delay (arendst#1486)
 * Add libraries
Adafruit_BME680-1.0.5, Adafruit_Sensor-1.0.2.02, TasmotaSerial-1.0.0 and
TSL2561-Arduino-Library
 * Add command Color2 to set color while keeping
same dimmer value
 * Add device function pointers
 * Add support for
SenseAir S8 CO2 sensor
 * Add color led signal to Carbon Dioxide (CO2)
sensors using defines CO2_LOW and CO2_HIGH in user_config.h
 * Add
support for Domoticz Air Quality sensor to be used by MH-Z19(B) and
SenseAir sensors
 * Add support for PZEM004T energy sensor
 * Add
support for iTead SI7021 temperature and humidity sensor by
consolidating DHT22 into AM2301 and using former DHT22 as SI7021 (arendst#735)

* Add support for BME680 using adafruit libraries (arendst#1212)
 * Add support
for MH-Z19(B) CO2 sensor (arendst#561, arendst#1248)
 * Add multipress support and
more user configurable GPIO to Sonoff Dual R2 (arendst#1291)
 * Add support for
TSL2561 using adafruit library (arendst#661, arendst#1311)
 * Add support for SHT3x
(arendst#1314)
 * Add support for Arilux LC06 (arendst#1414)
 * Add Italian language
file (arendst#1449)
 * Add 2nd Gen Alexa support to Wemo emulation discovery
(arendst#1357, arendst#1450)
 * Add define for additional number of WS2812 schemes
(arendst#1463)
@reloxx13 reloxx13 closed this May 15, 2018
@reloxx13 reloxx13 deleted the feature/Discover_2G_Fix branch May 15, 2018 20:18
curzon01 pushed a commit to curzon01/Tasmota that referenced this pull request Sep 6, 2018
5.10.0h
 * Fix Wemo Emulation for Gen 2 devices (arendst#1486)
curzon01 pushed a commit to curzon01/Tasmota that referenced this pull request Sep 6, 2018
5.10.0i
 * Add device function pointers
 * Moved Command and JSON
English language defines to one file (i18n.h)
 * Update it-IT.h language
file (arendst#1449)
 * Fix Wemo and Hue emulation by adding M-Search response
delay (arendst#1486)
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

Successfully merging this pull request may close these issues.

3 participants