Skip to content

session.h: Support building with LwIP, but LWIP_IPV4 or LWIP_IPV6 not set#264

Merged
obgm merged 1 commit into
eclipse-tinydtls:mainfrom
mrdeep1:lwip_no_ipv6
Nov 19, 2025
Merged

session.h: Support building with LwIP, but LWIP_IPV4 or LWIP_IPV6 not set#264
obgm merged 1 commit into
eclipse-tinydtls:mainfrom
mrdeep1:lwip_no_ipv6

Conversation

@mrdeep1

@mrdeep1 mrdeep1 commented Nov 14, 2025

Copy link
Copy Markdown
Contributor

Add in a dummy struct sockaddr_in6.

@mrdeep1

mrdeep1 commented Nov 14, 2025

Copy link
Copy Markdown
Contributor Author

Just done a re-validate of eclipsefdn/eca.

@boaks

boaks commented Nov 15, 2025

Copy link
Copy Markdown
Contributor

Just done a re-validate of eclipsefdn/eca.

"There is a valid ECA on file for supjps-libcoap ..."

Works now.

LGTM.

@boaks

boaks commented Nov 15, 2025

Copy link
Copy Markdown
Contributor

@obgm

Please a fast go :-).

@obgm obgm left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am not really fond of using non-standard types like u8_t and u32_t instead of the standard types. As this seems to be for LWIP only, I can live with that.

Comment thread session.h
Add in a dummy struct sockaddr_in or sockaddr_in6 as appropriate as
would be defined in LwIP.

Signed-off-by: Jon Shallow <supjps-libcoap@jpshallow.com>
@mrdeep1 mrdeep1 changed the title session.h: Support building with LwIP, but LWIP_IPV6 not set session.h: Support building with LwIP, but LWIP_IPV4 or LWIP_IPV6 not set Nov 19, 2025
@mrdeep1

mrdeep1 commented Nov 19, 2025

Copy link
Copy Markdown
Contributor Author

Added in dummy struct sockaddr_in if LWIP_IPV4 not set.

@obgm obgm merged commit 87cd057 into eclipse-tinydtls:main Nov 19, 2025
6 checks passed
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