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

+CWLAP Response contains brackets which break the API convention #856

Open
3 tasks done
JHedgesBartec opened this issue Jul 19, 2024 · 3 comments
Open
3 tasks done

Comments

@JHedgesBartec
Copy link

JHedgesBartec commented Jul 19, 2024

Answers checklist

  • I have read the documentation ESP-AT Programming Guide and the issue is not addressed there.
  • I have used the latest released firmware or have updated my ESP-AT branch (master or release) to the latest version and checked that the issue is present there.
  • I have searched the issue tracker for a similar issue and not found a similar issue.

AT+GMR

Source built from v3.2.0.0 release branch with BT Classic enabled, hence the modified BIN version.

AT+GMR
AT+GMR
AT version:3.2.0.0(s-ec2dec2 - ESP32 - Jul 28 2023 07:05:28)
SDK version:v5.0.2-376-g24b9d38a24-dirty
compile time(5c1d4b67):Jul 16 2024 17:06:45
Bin version:v4.0.0.0(WROOM-32)

OK

ESP-AT Firmware Source

v3.2.0.0 release branch, compiled locally to enable BT Classic on the ESP32-WROOM-32

Hardware Information

ESP32-WROOM-32

Power Supply used

USB

What is the expected behavior?

The +CWLAP response conforms to the documented format here

+CWLAP:<ecn>,<ssid>,<rssi>,<mac>,<channel>,<freq_offset>,<freqcal_val>,<pairwise_cipher>,<group_cipher>,<bgn>,<wps>

What is the actual behavior?

The +CWLAP response contains () brackets around all parameters. This does not match the documentation and breaks the general API format of <cmd>: followed by comma separated parameters. Having the parameters bracketed means it does not conform to the format of other API commands and has to be custom parsed.

Example:
+CWLAP:(3,"TEST_NETGEAR26",-54,"22:6d:34:30:7f:72",11,-1,-1,4,4,7,0)

Probability of recurrence

100% repeatable

AT+SYSRAM?

AT+SYSRAM?
AT+SYSRAM?
+SYSRAM:151140,144872
OK

Steps to reproduce

Initialise Wi-Fi in station mode with AT+CWMODE=1 and then perform a list of access points using AT+CWLAP

AT command port output

AT+GMR
AT+GMR
AT version:3.2.0.0(s-ec2dec2 - ESP32 - Jul 28 2023 07:05:28)
SDK version:v5.0.2-376-g24b9d38a24-dirty
compile time(5c1d4b67):Jul 16 2024 17:06:45
Bin version:v4.0.0.0(WROOM-32)

OK
AT+CWMODE=1
AT+CWMODE=1

OK
AT+CWLAP
AT+CWLAP
+CWLAP:(3,"TEST_NETGEAR26",-52,"22:6d:34:30:7f:72",11,-1,-1,4,4,7,0)

OK
AT+SYSRAM?
AT+SYSRAM?
+SYSRAM:151140,144872
OK

AT log port output

Compiled with silence mode so no log output.

More Information.

No response

@JAndrassy
Copy link

it is this way a very long time now. it would be a breaking change if the {} are removed

@JHedgesBartec
Copy link
Author

Authorised personnel should then update the documentation to reflect actual behaviour

@ustccw
Copy link
Collaborator

ustccw commented Aug 7, 2024

@JHedgesBartec @JAndrassy Thanks a lot! We should update the ESP-AT document.

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

No branches or pull requests

3 participants