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

Fix path for rpcbind defaults and fix rpcbind option #3033

Merged
merged 2 commits into from
Oct 10, 2023

Conversation

ricardobranco777
Copy link
Contributor

Fix path for rpcbind defaults and fix rpcbind option.

Fix path for rpcbind defaults and fix rpcbind option.
@lurch
Copy link
Contributor

lurch commented Jul 31, 2023

It looks like this file now says:

# /etc/init.d/rpcbind

OPTIONS=""

# Cause rpcbind to do a "warm start" utilizing a state file (default)
OPTIONS="-w"

# Uncomment the following line to restrict rpcbind to localhost only for UDP requests
# OPTIONS="${OPTIONS} -h 127.0.0.1 -h ::1"

# Uncomment the following line to enable libwrap TCP-Wrapper connection logging
# OPTIONS="${OPTIONS} -l "

so the bits talking about "to get rid of either -l or -h 127.0.0.1" don't seem correct?

Ping @pelwell in case he has any NFS-knowledge (or knows who does?)

@pelwell
Copy link
Contributor

pelwell commented Jul 31, 2023

@popcornmix is the main NFS user here.

@pelwell
Copy link
Contributor

pelwell commented Jul 31, 2023

At first glance that whole line now appears to be redundant.

@popcornmix
Copy link
Contributor

I only run nfs shares on a trusted network so I haven't looked at securing ports with rpcbind.

Typically I just install nfs-kernel-server / nfs-common, and edit /etc/exports / /etc/fstab and things work for me.
This seems far more complicated (presumably for security reasons).

@aallan aallan changed the title Update nfs.adoc Fix path for rpcbind defaults and fix rpcbind option Jul 31, 2023
@aallan aallan requested a review from jehlers42 August 25, 2023 15:22
@aallan
Copy link
Contributor

aallan commented Aug 25, 2023

@jehlers42 Can you look at this one, and get it ready to merge, or close as appropriate.

@aallan
Copy link
Contributor

aallan commented Sep 6, 2023

Actually @nathan-contino this might be one for you!

@nathan-contino
Copy link
Collaborator

@aallan removed this line as per discussion with @popcornmix. Ready to merge when you are.

@aallan aallan merged commit 80bc371 into raspberrypi:develop Oct 10, 2023
1 check passed
@aallan
Copy link
Contributor

aallan commented Oct 10, 2023

Merged!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants