-
-
Notifications
You must be signed in to change notification settings - Fork 26
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add DnsAddress to rama-net #360
Comments
Hello Glen, i just wanted to say thank you and i'm interested in contributing to this, how can i get started ? |
You could take a similar approach as the contributor for #358 is taking. Together with that as inspiration and this issues's description I would hope you have enough information. But if not, do let me know with a specific set of questions you have and I'll gladly answer these. |
Hey! I'm interested in working on this issue. |
Let's give @Muktarsadiq a day more to respond as he seemed to want to pick this up as his first contribution. Might be that life got in the way of it or that he changed his mind as I haven't heard back from him since. I'll assign it to you tomorrow @Hrushi20 in case he didn't come back yet with a reply or update tomorrow. But feel free to open a PR already if you can't wait, look around the code base, make another contribution or ask for guidance or feedback in meanwhile. Thx for the interest to participate! |
@Hrushi20 feel free to start working on it, it's all yours now. @Muktarsadiq in case you were temporarily out, I hope you understand. There's plenty of other fun stuff that can be done though. Feel free to reach out to me in another ticket, discord or email. |
Thank you. Looking into it. |
Is the goal to add ip_addr and port to |
The goal is to have a type that you can name This type is similar to the The reason to want to have this type is because there are situations where you want to clearly communicate that you are using a domain+host pair. And it would be silly if everyone using rama needs to do that for themselves. An So therefore we need a new type. To start you can copy basically Do let me know if you need more guidance. |
It's pretty much like Authority from the same rama-net package (https://ramaproxy.org/docs/rama/net/address/index.html)
But it is a Domain Port (u16) pair, so Domain instead of Host.
This type is convenient if you want to enforce that a Domain is used without the (type) possibility of it also being an IP.
This is a twin-issue of #358.
The text was updated successfully, but these errors were encountered: