Skip to content

Conversation

@daffainfo
Copy link
Contributor

Template / PR Information

  • Fixed CVE-2020-XXX / Added CVE-2020-XXX / Updated CVE-2020-XXX
  • References:

Template Validation

I've validated this template locally?

  • YES
  • NO

Additional Details (leave it blank if not applicable)

Additional References:

@DhiyaneshGeek DhiyaneshGeek self-assigned this Jun 10, 2025
@DhiyaneshGeek
Copy link
Member

Hi @daffainfo

Is there any specific reason to remove these ports ?

@daffainfo
Copy link
Contributor Author

daffainfo commented Jun 10, 2025

Hello @DhiyaneshGeek

Usually the BaseURL variable already contains the port, right? for example like

https://example.com:1337

If we hardcoded the port it will broke the template

https://example.com:1337:4040

If you want to hardcode the port, add more line or just change the current line, for example like i did to this template or something like this

- "{{BaseURL}}/jobs/"
- "{{Host}}:4040/jobs/"

@DhiyaneshGeek DhiyaneshGeek added Done Ready to merge and removed waiting for more info labels Jun 16, 2025
@ritikchaddha ritikchaddha merged commit 9ff744e into projectdiscovery:main Jun 24, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Done Ready to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants