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

decode static URI before serving #754

Merged
merged 5 commits into from
Jun 2, 2022

Conversation

meztez
Copy link
Collaborator

@meztez meztez commented Jan 7, 2021

Closes #753

PR task list:

  • Update NEWS
  • Add tests
  • Update documentation with devtools::document()

@meztez
Copy link
Collaborator Author

meztez commented Jan 7, 2021

I'm thinking of creating the file on the flies for the test. Since I get an R check error for the filename.

@schloerke
Copy link
Collaborator

Will need to check if file exists as some systems won't be able to write chinese characters. 😞

Or maybe skip on cran and only test on macos? Then the logic will at least be tested inside GHA.

@meztez
Copy link
Collaborator Author

meztez commented Jan 7, 2021

I'll wait to see which one fails

@meztez
Copy link
Collaborator Author

meztez commented Jan 7, 2021

so skip on CRAN

@schloerke schloerke added this to the v1.2.0 milestone Jan 18, 2021
@schloerke schloerke merged commit f96422b into rstudio:main Jun 2, 2022
schloerke added a commit that referenced this pull request Jun 2, 2022
* main:
  Fix Content-Length header on HTTP responses which forbid it (#760)
  Generate docs and fix existing tests (#860)
  decode static URI before serving (#754)
  Update feather serializer; Add parquet serializer (#849)
  remove author in pkgdown; have website build again
  Fix plumber website. Use tidytemplate (#851)
  Runtime Vignette typo (#848)
  GHA v2 (#842)
  master -> main (#839)
  Create GeoJSON serializer and parser (#830)
  Add details about named functions as router modifiers (#824)
  Parser params should be in a `list()` (#827)
  Test on ubuntu 18/20 and use relative R versions (#828)
  Shared secret rejections are not errors (#809)
  Fix tag endpoint block annotation spelling to `@tag` (#800)
  Add ORCID info (#794)
  Add blog post link to website (#792)
  Use serializer Content-Type header for PlumberEndpoint API spec (#789)
  Use dev version (#790)
  v1.1.0 (#752)
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.

Maybe support Chinese character in req$PATH_INFO
2 participants