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

Create noauthed master for mirror publiclist #80

Merged
merged 1 commit into from May 19, 2015
Merged

Conversation

puiterwijk
Copy link
Contributor

This makes it possible to make publiclist varnish cached
because that doesn't depend on any cookie status anymore.

Signed-off-by: Patrick Uiterwijk puiterwijk@redhat.com

@adrianreber
Copy link
Member

I cannot say if it is correct but I like the idea very much to be able to cache the mirrorlist as the content does not change that often.

@pypingou
Copy link
Member

Does not compile:

mirrormanager2/mirrormanager2/templates/fedora/master.html", line 3, in template
    {% block nav %}
TemplateSyntaxError: unexpected '%'

This makes it possible to make publiclist varnish cached
because that doesn't depend on any cookie status anymore.

Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
@puiterwijk
Copy link
Contributor Author

The compile error is now fixed :)

@pypingou
Copy link
Member

Cool, testing it looks good to me! :)

@pypingou
Copy link
Member

This way, we can cache everything but the front page (and the pages needing auth), nice and easy! :)

puiterwijk added a commit that referenced this pull request May 19, 2015
Create noauthed master for mirror publiclist
@puiterwijk puiterwijk merged commit b2c28bf into master May 19, 2015
@puiterwijk
Copy link
Contributor Author

Thanks, this should improve it a lot!

@puiterwijk puiterwijk deleted the split-publiclist branch May 19, 2015 16:26
pypingou added a commit that referenced this pull request May 20, 2015
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.

None yet

3 participants