Skip to content

Fix typo in nginx redirect for #111 knowledgebase URL - #1075

Merged
nathanrod merged 1 commit into
ivpn:mainfrom
lamtrinhdev:Fix_Direct_How-many-IPs-do-you-have-in-total-or-per-server.html
Oct 18, 2026
Merged

nathanrod merged 1 commit into
ivpn:mainfrom
lamtrinhdev:Fix_Direct_How-many-IPs-do-you-have-in-total-or-per-server.html

Conversation

@lamtrinhdev

Copy link
Copy Markdown
Contributor

PR type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes
  • Other... Please describe:

Description:

The redirect for the old /knowledgebase/111/How-many-IPs-do-you-have-in-total-or-per-server.html URL was pointing to /knowledgebase/genera/, which doesn't exist — so anyone following that old link would end up on a 404 page.

This changes the target to /knowledgebase/general/, matching every other rule in that block.

  -    rewrite ^/knowledgebase/111/How-many-IPs-do-you-have-in-total-or-per-server.html$ /knowledgebase/genera/ permanent;
  +    rewrite ^/knowledgebase/111/How-many-IPs-do-you-have-in-total-or-per-server.html$ /knowledgebase/general/how-many-ips-do-you-have-in-total-or-per-server/ permanent;

How to test:

copy url https://www.ivpn.net/knowledgebase/111/How-many-IPs-do-you-have-in-total-or-per-server.html to browser, it displays 404

Expected:

https://www.ivpn.net/knowledgebase/111/How-many-IPs-do-you-have-in-total-or-per-server.html will redirect to https://www.ivpn.net/knowledgebase/general/how-many-ips-do-you-have-in-total-or-per-server/

@lamtrinhdev

Copy link
Copy Markdown
Contributor Author

Dear @johnnyburnaway ,

I have a minor update for redict page from old url to new url. When you have time, please help me to review it.

Thanks,
Lam

@nathanrod
nathanrod merged commit eb71739 into ivpn:main Oct 18, 2026
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