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

Retain quotes on relative URLs in CSS #19

Closed

Conversation

wilkinsona
Copy link
Contributor

cssnano's normalizeUrl removes quotes from relative URLs, underminining the fix for #11 that was made in 8aab4d7.

There's no configuration option that I can see on normalizeUrl to disable the removal of quotes, so this PR disables normalizeUrl entirely.

cssnano's normalizeUrl removes quotes from relative URLs, underminining
the fix for spring-io#11 that was made in 8aab4d7.

There's no configuration option that I can see on normalizeUrl to
disable the removal of quotes, so this commit disables normalizeUrl
entirely.
@philwebb philwebb added this to the 0.0.1 milestone Mar 11, 2021
philwebb pushed a commit that referenced this pull request Mar 11, 2021
cssnano's normalizeUrl removes quotes from relative URLs, underminining
the fix for #11 that was made in 8aab4d7.

There's no configuration option that I can see on normalizeUrl to
disable the removal of quotes, so this commit disables normalizeUrl
entirely.

See gh-19
@philwebb philwebb closed this in d0e0eb4 Mar 11, 2021
@wilkinsona wilkinsona deleted the retain-quotes-on-css-urls branch March 11, 2021 16:47
philwebb pushed a commit to philwebb/spring-asciidoctor-backends that referenced this pull request Jul 7, 2021
Prior to this commit, we introduced a basic implementation for GraphQL
metrics with Spring GraphQL. The main problem was that we were tracking
the GraphQL queries themselves and those are good candidates for
tags cardinality explosion.

This commit revisits the metrics arrangement and introduces:

* `graphql.request` as the main metric; this tracks the execution of a
  single request (i.e. multiple queries)
* `graphql.datafetcher` for tracking data fetching operations.
* `graphql.error` for counting errors.

Closes spring-iogh-19
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.

2 participants