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

websocket client build failed with esp-idf version 5.1 (IDFGH-10123) #294

Closed
3 tasks done
hoangdovan opened this issue May 14, 2023 · 4 comments
Closed
3 tasks done

Comments

@hoangdovan
Copy link

Answers checklist.

  • I have read the documentation for esp-protocols components and the issue is not addressed there.
  • I have updated my esp-protocols 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.

What component are you using? If you choose Other, provide details in More Information.

esp_websocket_client

component version

No response

IDF version.

v5.1-dev-2356-gedd815af2e-dirty

More Information.

Hello, I have just try to compile esp_websocket_client component with esp-idf v5.1. Then there are some error happen:

managed_components/espressif__esp_websocket_client/esp_websocket_client.c:448:14: error: 'esp_transport_ws_config_t' has no member named 'auth'
448 | .auth = client->config->auth,
| ^~~~
managed_components/espressif__esp_websocket_client/esp_websocket_client.c:449:41: warning: initialized field overwritten [-Woverride-init]
449 | .propagate_control_frames = true
| ^~~~
/managed_components/espressif__esp_websocket_client/esp_websocket_client.c:449:41: note: (near initialization for 'config.propagate_control_frames')
/managed_components/espressif__esp_websocket_client/esp_websocket_client.c: In function 'esp_websocket_client_task':
/managed_components/espressif__esp_websocket_client/esp_websocket_client.c:849:70: error: implicit declaration of function 'esp_transport_ws_get_upgrade_request_status' [-Werror=implicit-function-declaration]
849 | client->error_handle.esp_ws_handshake_status_code = esp_transport_ws_get_upgrade_request_status(client->transport);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

I think esp_websocket_client is not compatible with esp-idf version 5.1 yet?

@github-actions github-actions bot changed the title websocket client build failed with esp-idf version 5.1 websocket client build failed with esp-idf version 5.1 (IDFGH-10123) May 14, 2023
@david-cermak
Copy link
Collaborator

@hoangdovan Which version of the component are you using? The current latest version is v1.0.0 (https://components.espressif.com/components/espressif/esp_websocket_client) and it should work with both v5.0 and v5.1 IDF releases.

@hoangdovan
Copy link
Author

@david-cermak Hello David, I use latest version v1.0.0, and esp-idf version v5.1-dev-2356-gedd815af2e-dirty (master branch). And cannot compile because some following problems:

  1. esp_transport_ws_config_t does not have "auth" property member
  2. esp_transport_ws_get_upgrade_request_status() function does not exist

I doubt that with esp-idf version 5.1 some breaking change cause this problem happen.
I will check again about this matter.
Thank you!

@david-cermak
Copy link
Collaborator

FYI: just referencing the recent CI job building our websocket example with IDF v5.1: https://github.com/espressif/esp-protocols/actions/runs/5004207678/jobs/8966478990#step:4:55

(note that this is the latest version of the component, not v1.0.0, but I see only cosmetic/docs changes in between)

@hoangdovan
Copy link
Author

@david-cermak I already confirmed again with esp-idf version 5.2, and I can compile without any error. Thank you very much!

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