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

302 Found #21

Open
JohanSF opened this issue Dec 15, 2019 · 4 comments
Open

302 Found #21

JohanSF opened this issue Dec 15, 2019 · 4 comments
Assignees

Comments

@JohanSF
Copy link

JohanSF commented Dec 15, 2019

Hi there, I just tried implementing this but i get 302 Found error:
image
Im running LE/nginx with this container: https://hub.docker.com/r/linuxserver/letsencrypt/

@Naereen Naereen self-assigned this Dec 16, 2019
@Naereen
Copy link
Owner

Naereen commented Dec 16, 2019

Hi @JohanSF,
Thanks for trying this package.
I won't have access to a machine able to run this docker container, until a few weeks, so I'm sorry but cannot try to replicate your issue.

Are you able to give me more information?

I do not know what to tell you except that a 302 error indicates a redirection to follow.
Maybe the solution is to change the javascript code that fetches the HEADER.md and README.md files, in order to follow this redirection?

@JohanSF
Copy link
Author

JohanSF commented Dec 17, 2019

This is without the theme:
image
The code is this:

location /newsmags {
	fancyindex on;              # Enable fancy indexes.
	fancyindex_localtime on;
	fancyindex_exact_size off;  # Output human-readable file sizes.
	fancyindex_header "/config/www/Nginx-Fancyindex-Theme-dark/header.html";
	fancyindex_footer "/config/www/Nginx-Fancyindex-Theme-dark/footer.html";
	#fancyindex_ignore "examplefile.html";
	fancyindex_name_length 255;
	auth_request /auth-3; #For server-auth with Organizr, remove unless you use Organizr.
	alias /config/www/newsmags;
	include /config/nginx/proxy.conf;
}

I tinkered around and found that no matter what I set the fancyindex_header to, for instance a file that does not exist, I get the 302 Found error :S but I am sure the path is correct, and it is confirmed by that fact that the alias command works. And even further proof:
image
The files from cloning the repo:
image
And I also tried with the release 1.1, neither seems to work for me. I am stumped.

@JohanSF
Copy link
Author

JohanSF commented Dec 17, 2019

I see you put the bug tag on, but your theme is not the source of the error. As I get the error no matter what I set the header or footer to.

@Naereen Naereen removed the bug label Dec 17, 2019
@Naereen
Copy link
Owner

Naereen commented Dec 17, 2019

Thanks for the feedback. I don't really know…

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

No branches or pull requests

2 participants