Skip to content
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

XOVER Support #4

Open
kevingy opened this issue Aug 31, 2020 · 2 comments · Fixed by #6 or #7
Open

XOVER Support #4

kevingy opened this issue Aug 31, 2020 · 2 comments · Fixed by #6 or #7
Assignees
Labels
enhancement New feature or request

Comments

@kevingy
Copy link
Contributor

kevingy commented Aug 31, 2020

The Usenet server that I am using does not support the OVER command. It only supports the older XOVER command. For backwards compatibility, perhaps this library should also support XOVER. The syntax is identical to OVER, except for the additional X in XOVER.

I've already coded XOVER by copying OVER and adding the X. There is certainly a more elegant way to implement XOVER without copying the entire method. If this is something valuable to the project, I will create the more elegant solution and create a pull request.

@seanmcelroy seanmcelroy added the enhancement New feature or request label Aug 31, 2020
@seanmcelroy
Copy link
Owner

I'd be happy to take a PR. Is the server publicly accessible? I'd like to dig in a little deeper to understand other features in its capabilities that may also need to be mapped.

@kevingy
Copy link
Contributor Author

kevingy commented Aug 31, 2020

It is a commercial server - astraweb.com. XOVER is an older method from 2000. The X specifies an extended or experimental method. XOVER was replaced by OVER IN 2006, but XOVER is still widely used.

kevingy added a commit to kevingy/nntp-client that referenced this issue Sep 2, 2020
@seanmcelroy seanmcelroy self-assigned this Sep 7, 2020
This was linked to pull requests Sep 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
2 participants