Support P2P Connections with Custom Signaling Between Steam and OSS Library Versions
Nobody has claimed this yet.
Assessment
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Newbie friendliness
- 38/100
- Issue type
- Feature
- Clarity
- Mostly clear
- Activity status
- Stale
- Tech stack
- cpp
- Domain
- networking, security
Research direction
Start with CSteamNetworkConnectionBase::AllowRemoteUnsignedCert and the NetInterfaceGNS.cpp:188 warning path. Trace how networking configuration variables are defined and applied, focusing on the proposed k_ESteamNetworkingConfig_P2P_AllowWithoutAuth setting for non-SDR connections. Done means the setting controls whether these unsigned P2P connections are accepted between Steam and OSS library versions.
Written by the indexing model from the issue text.
Description
As a user, I would like to establish P2P connections using custom signaling between the Steam version of the library and the OSS (Open Source Software) version.
However, the Steam version currently disallows unauthenticated/unsigned P2P communication, which prevents this interoperability.
EUnsignedCert CSteamNetworkConnectionBase::AllowRemoteUnsignedCert()
{
#ifdef STEAMNETWORKINGSOCKETS_OPENSOURCE
// We don't have a cert authority. We probably ought to make this customizable
return k_EUnsignedCert_Allow;
#else
return k_EUnsignedCert_Disallow;
#endif
}
[11:35:39.700] [warning] NetInterfaceGNS.cpp:188 - Failed to start accepting P2P connect request from str:393FD8571292475BB1128DCC7D73745 on vport 0; Unsigned certs are not allowed
Ideally it could be controlled with new config var for non-SDR connections.
k_ESteamNetworkingConfig_P2P_AllowWithoutAuth
- Dominant language
- C++
- Stars
- 10k
- Forks
- 747
- Avg merge
- 5d 19h
- Merged PRs (30d)
- 1
Getting set up
This project ships no dev container, Dockerfile or contributing guide, so setting up is up to you: start from its README, and see our first-contribution guide for the general steps.
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from ValveSoftware/GameNetworkingSockets
-
Native ICE resolves STUN/TURN hostnames under the global lockPossibly taken @tintinhamans claimed this 11 days ago. Open
Difficulty 4/5 3-5 days Newbie friendliness 55/100
-
WebRTC ICE keeps an idle thread per STUN/TURN hostname lookupPossibly taken @tintinhamans claimed this 11 days ago. Open
Difficulty 3/5 1-2 days Newbie friendliness 68/100
-
Difficulty 5/5 Over a week Newbie friendliness 25/100
ValveSoftware/GameNetworkingSockets#425 · 7 comments ·
-
steamworks
Difficulty 4/5 3-5 days Newbie friendliness 35/100
ValveSoftware/GameNetworkingSockets#386 · 2 comments ·
-
Difficulty 4/5 3-5 days Newbie friendliness 30/100
All issues in ValveSoftware/GameNetworkingSockets
Similar issues
-
feature request
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
Maintainers usually reply within 2 days
-
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
Maintainers usually reply within 1 day
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
gavinlouuu-kpt/mib-studio-qt#517 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
Maintainers usually reply within 1 day
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 73/100
EchoTools/nevr-runtime#116 ·
Maintainers usually reply within 1 day