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

Springclean #529

Merged
merged 4 commits into from
Oct 29, 2024
Merged

Springclean #529

merged 4 commits into from
Oct 29, 2024

Conversation

paulusmack
Copy link
Collaborator

Delete some old stuff of little value now that no-one uses dial-up modems any more. This will reduce the maintenance burden and scope for vulnerabilities a little.

This is prompted by a number of factors:

* It was more useful back in the dial-up days, but no-one uses dial-up
  any more

* In many cases there will be no terminal accessible to the prompter
  program at the point where the prompter is run

* The passwordfd plugin does much the same thing but does it more
  cleanly and securely

* The handling of privileges and file descriptors needs to be audited
  thoroughly.

Signed-off-by: Paul Mackerras <[email protected]>
This is associated with the passprompt plugin, and like it, seems not
to be very useful any more now that no-one uses dial-up.  Also, its
function seems somewhat peripheral to PPP.

Signed-off-by: Paul Mackerras <[email protected]>
This removes various scripts and config files that related to dial-up
connections, doing PPP over rsh or ssh (for which there are better
alternatives), and updating resolv.conf (for which distros have other
mechanisms these days).

Signed-off-by: Paul Mackerras <[email protected]>
The commentary and example in this file doesn't seem all that useful.

Signed-off-by: Paul Mackerras <[email protected]>
@paulusmack paulusmack merged commit 616102e into master Oct 29, 2024
61 checks passed
@paulusmack paulusmack deleted the springclean branch November 21, 2024 03:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant