Skip to content

Fix issue #3163: corrected the IP-domain rlation statement#3164

Open
EnTechGR wants to merge 1 commit into
01-edu:masterfrom
EnTechGR:fix-issue-3163
Open

Fix issue #3163: corrected the IP-domain rlation statement#3164
EnTechGR wants to merge 1 commit into
01-edu:masterfrom
EnTechGR:fix-issue-3163

Conversation

@EnTechGR
Copy link
Copy Markdown

CON-3163 | Corrected IP-Domain mapping documentation

Why?

The existing documentation contained a logical inaccuracy regarding the relationship between domain names and IP addresses. It stated that an IP address can only identify one domain name, which ignores the industry-standard practice of Virtual Hosting. This change ensures the technical documentation correctly reflects how modern web servers and DNS function.

Solution Overview

I have updated the descriptive text to clarify that the relationship between domain names and IP addresses is many-to-many.

The corrected statement now explains that:

  1. One Domain can resolve to Multiple IPs (for load balancing).

  2. One IP can host Multiple Domains (via Virtual Hosting/SNI).

Implementation Details

The revision replaces the restrictive "one-to-one" implication with a more accurate "many-to-many" explanation.

  • Approach Chosen: Directly addressing the "Virtual Hosting" concept. This was chosen over a simple deletion because explaining why the numbers match the names provides better educational value for the project.

  • Alternatives Considered: I considered adding a deep dive into SNI (Server Name Indication), but decided against it to keep the "Just Numbers" section concise and focused on the core concept of identifiers.

@motonai
Copy link
Copy Markdown
Contributor

motonai commented May 14, 2026

The "industry-standard of virtual hosting" has nothing to do with the fact that an IP can point only to one domain. For more info, read the reply at #3163 and its references.

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.

2 participants