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

Migrate from httr to httr2 #154

Open
5 tasks
florianm opened this issue Mar 16, 2024 · 0 comments
Open
5 tasks

Migrate from httr to httr2 #154

florianm opened this issue Mar 16, 2024 · 0 comments
Assignees
Labels
tech-debt Reduce technical debt

Comments

@florianm
Copy link
Collaborator

florianm commented Mar 16, 2024

Feature

  • ruODK uses httr for web requests.
  • httr is no longer developed, httr2 is the recommended replacement.
  • httr2 is pipe-friendly and looks like a major usability improvement over httr (which was already a really great tool).

Examples:

Approach

  • The user experience of ruODK should not change.
  • Exception: pass on improved httr2 error handling and messaging.
  • Factor out request handling into a centralised, single set of wrappers around httr.
  • Migrate every function to use the centralised helpers.
  • Introduce httr2 as dependency and migrate first function.
  • Migrate all functions.
  • Drop httr as dependency.
@florianm florianm added the feature a feature request or enhancement label Mar 16, 2024
@florianm florianm self-assigned this Mar 16, 2024
@florianm florianm added tech-debt Reduce technical debt and removed feature a feature request or enhancement labels Mar 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tech-debt Reduce technical debt
Projects
None yet
Development

No branches or pull requests

1 participant