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

the userdir apache module is install by default and with this change is configure u… #1682

Closed
wants to merge 1 commit into from

Conversation

c4m4
Copy link

@c4m4 c4m4 commented Aug 26, 2017

…serdir.conf with <Directory /*/> and Multiviews is begin applied to all dirs

This reverts commit e37b994, reversing
changes made to 0c69992.

…serdir.conf with <Directory /*/> and Multiviews is begin applied to all dirs

This reverts commit e37b994, reversing
changes made to 0c69992.
@c4m4
Copy link
Author

c4m4 commented Aug 26, 2017

If I using apache module with this change, the apache MultiViews option is applied to all dirs, If I don't set any dir for the userdir module the module configuration that puppet generate is as follow:
cat /etc/apache2/mods-enabled/userdir.conf

<IfModule mod_userdir.c>
  UserDir disabled root
  UserDir /home/*/public_html 

  <Directory "/*/"> # :(
    AllowOverride FileInfo AuthConfig Limit Indexes
    Options MultiViews Indexes SymLinksIfOwnerMatch IncludesNoExec
    <Limit GET POST OPTIONS>
      Order allow,deny
      Allow from all
    </Limit>
    <LimitExcept GET POST OPTIONS>
      Order allow,deny
      Allow from all
    </LimitExcept>
  </Directory>
</IfModule>

@c4m4 c4m4 changed the title the userdir is install by default and with this change is configure u… the userdir apache module is install by default and with this change is configure u… Aug 26, 2017
@eputnam
Copy link
Contributor

eputnam commented Aug 28, 2017

@c4m4 welcome to github! should Directory match the UserDir path? "/home/*/public_html" from the example above. see #1683

@c4m4
Copy link
Author

c4m4 commented Aug 29, 2017

yes, if any parameters was given the path need to be /home/*/public_html, Directory = UserDir

@eputnam
Copy link
Contributor

eputnam commented Aug 29, 2017

@c4m4 thanks for bringing this to our attention! the ability to pass a custom path is a feature many users have requested so I'm hesitant to revert it. Instead, i believe #1683 will make both parties happy so I'm going to close this. Please feel free to re-open if you feel strongly about this feature being reverted.

@eputnam eputnam closed this Aug 29, 2017
@c4m4
Copy link
Author

c4m4 commented Aug 30, 2017

I solved removing userdir module from my setup, by I think the module need to generate UserDir /home/*/public_html and <Directory "/home/*/public_html"> too, in this way, by default MultViews isn't apply to all apache dirs and the users can specify a costum userdir too.

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

Successfully merging this pull request may close these issues.

2 participants