Skip to content

Fix typo in symfony service filename. #217

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

Merged
merged 2 commits into from
Jun 8, 2015
Merged

Fix typo in symfony service filename. #217

merged 2 commits into from
Jun 8, 2015

Conversation

chtipepere
Copy link
Contributor

Using symfony as proxy_client failed.

@chtipepere
Copy link
Contributor Author

Hmm... Not really sure if this is the issue I'm facing...
I close this PR for the moment.

@chtipepere chtipepere closed this Jun 8, 2015
@dbu
Copy link
Contributor

dbu commented Jun 8, 2015

looking at https://github.com/FriendsOfSymfony/FOSHttpCacheBundle/tree/master/Resources/config i think you are correct. and i am quite surprised our tests don't show this problem.

can you please reopen the pull request? i want to merge it.

@chtipepere
Copy link
Contributor Author

I think there are other issues...
I'll re-open the PR and update it.

@chtipepere chtipepere reopened this Jun 8, 2015
@dbu
Copy link
Contributor

dbu commented Jun 8, 2015

oh, thanks for that one. do you see other issues or is that about it? did you get things to work in your application with this? its all quite new - but obviously we lack some configuration testing here.

@chtipepere
Copy link
Contributor Author

For the moment, I don't know if it's enough.
I'm just trying to use this bundle using symfony reverse proxy on sf2.3
I'll update the PR if I find something else, or add a comment if it's enough.

dbu added a commit that referenced this pull request Jun 8, 2015
Fix typo in symfony service filename.
@dbu dbu merged commit f9947c4 into FriendsOfSymfony:master Jun 8, 2015
@dbu
Copy link
Contributor

dbu commented Jun 8, 2015

please do a new pull request if you have more things. the fixes you had already are valid and i merged them. please ping when you are good, i will tag a new version then.

@chtipepere
Copy link
Contributor Author

I've finally been able to make the bundle works, but:

  • I have to disable tags in config (maybe add this to the documentation)
    tags:
        enabled:      false

Disabling the tags, I've got an other issue:

ServiceNotFoundException: The service "fos_http_cache.command.invalidate_tag" has a dependency on a non-existent service "fos_http_cache.handler.tag_handler".

(everything works fine when I comment the service)
I'll dig this issue today

@chtipepere
Copy link
Contributor Author

I'm trying something.
I'll send a new PR in a couple of hours.

@dbu
Copy link
Contributor

dbu commented Jun 8, 2015

hm. indeed the symfony cache for now does not support cache tagging. but this is inconvenient for the user.

in symfony cmf, we have some of these options as true|false|"auto" and auto means the feature is enabled if it makes sense. that would make sense for the tags. then with symfony or nginx, the whole tag stuff is not enabled, but with varnish it is enabled by default. users can change that to explicit true or false, and in the DI we could catch situations where its configured to true but the selected proxy client does not support that feature and give a better error message than a random servicenotfoundexception. wdyt?

@chtipepere
Copy link
Contributor Author

Well.

I've done some stuff to split tag_handler command service from the cache_manager.xml, and I do not have any Exception.
But cache is not invalidate.

Do I miss something, somewhere?

@dbu
Copy link
Contributor

dbu commented Jun 8, 2015

you mean you send purge requests to your symfony cache but it does not work? did you set up the HttpCache as explained in the documentation?

@chtipepere
Copy link
Contributor Author

Ok, I've find my problem.
The URL I tried to purge was not well written.
I'll open a new PR with the fix I've made locally to make it works.

@dbu
Copy link
Contributor

dbu commented Jun 8, 2015

thanks a lot!

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.

3 participants