Skip to content

URL-SYNTAX.md: fix port number mistakes for IMAP and LDAP - #20679

Closed
bagder wants to merge 1 commit into
masterfrom
bagder/url-port-nums
Closed

URL-SYNTAX.md: fix port number mistakes for IMAP and LDAP#20679
bagder wants to merge 1 commit into
masterfrom
bagder/url-port-nums

Conversation

@bagder

@bagder bagder commented Feb 22, 2026

Copy link
Copy Markdown
Member

Reported-by: Augment code
URL: #20673 (review)

@bagder
bagder marked this pull request as ready for review February 22, 2026 22:56
@bagder
bagder requested a review from Copilot February 22, 2026 22:57

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR corrects documentation errors for default port numbers in the URL-SYNTAX.md file. The incorrect port numbers for IMAP (132) and LDAP (369) are fixed to their correct values (143 and 389 respectively), which match both the IANA standard port assignments and curl's implementation in lib/urldata.h.

Changes:

  • Corrected IMAP default port from 132 to 143
  • Corrected LDAP default port from 369 to 389

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@bagder bagder closed this in 57a1cc5 Feb 22, 2026
@bagder
bagder deleted the bagder/url-port-nums branch February 22, 2026 23:01
outcast36 pushed a commit to greearb/curl that referenced this pull request Jun 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants