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.

Sign up for GitHub

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

Jump to bottom

pppd/lcp: lcp-up and lcp-down scripts. #342

Closed
wants to merge 1 commit into from
Closed

pppd/lcp: lcp-up and lcp-down scripts. #342

wants to merge 1 commit into from

Conversation

jkroonza
Copy link
Contributor

Signed-off-by: Jaco Kroon [email protected]

@jkroonza jkroonza mentioned this pull request May 17, 2022
Closed
@enaess
Copy link
Contributor

enaess commented May 31, 2022

@jkroonza - Could you please inform us of the use-case for this change?

@pali
Copy link
Contributor

pali commented May 31, 2022

I do not see need to have this script? Really what is the purpose of it?

@enaess
Copy link
Contributor

enaess commented May 31, 2022

@pali I did checked out the issue, #343 and it is firewall related. I still don't see a particular reason to take it.

@Neustradamus Neustradamus requested a review from paulusmack July 30, 2022 03:30
This was referenced Aug 23, 2022
Closed
Merged
@jkroonza
Copy link
Contributor Author

Superceded by #367

@jkroonza jkroonza closed this Aug 24, 2022
@jkroonza jkroonza deleted the lcp-updown branch December 4, 2023 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@paulusmack paulusmack Awaiting requested review from paulusmack

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jkroonza @enaess @pali