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

Fix UPF nodeID parsing issue when using FQDN #6

Closed
wants to merge 1 commit into from

Conversation

abousselmi
Copy link

@abousselmi abousselmi commented Sep 30, 2020

Hello @free5gc-org,

When an FQDN is used as UPF node_id in SMF configuration, IP parsing fails and PFCP session is not established between SMF and UPF. To resolve this issue, and since there is no native way to check whether node_id is IPv4, IPv6 or an FQDN, we need to try a name resolution before parsing the IP.

This issue was experienced in the docker version of free5gc: free5gc/free5gc-compose#9. This also provides a fix to issue free5gc/free5gc#111.

Thanks to @hhoai we now know the root cause of the problem and the solution. The verification step will make sure that the nodeID is updated only when the DNS resolution succeeds.

Cheers,

When an FQDN is used as UPF node_id in SMF configuration, IP parsing fails 
and PFCP session is not established between SMF and UPF. To resolve this 
issue, and since there is no native way to check whether node_is is IPv4, IPv6 
or an FQDN, we need to try a name resolution before parsing the IP.

This issue was experienced in the docker version of free5gc: free5gc/free5gc-compose#9
Thanks to @hhoai we now know the root cause of the problem and
the solution. The verification step will make sure that the nodeID is
updated only when the DNS resolution succeeds.
@abousselmi
Copy link
Author

As per the suggestion of @sh-sekigawa, resolving the NodeID as suggested here will make the FQDN case useless which is not the intended behaviour. The NodeIdType will never be a pfcpType.NodeIdTypeFqdn as the resolution happened before assigning the type.

@abousselmi abousselmi closed this Oct 5, 2020
free5gc-org pushed a commit that referenced this pull request Jan 18, 2021
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.

1 participant