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

bug: not able to change max body size #68

Closed
kennedymwavu opened this issue Jul 17, 2024 · 0 comments · Fixed by #69
Closed

bug: not able to change max body size #68

kennedymwavu opened this issue Jul 17, 2024 · 0 comments · Fixed by #69

Comments

@kennedymwavu
Copy link
Contributor

reprex

app <- Ambiorix$new()
app$limit <- 25 * 1024 * 1024

the reprex above throws this error:

Error in (function (value)  : object 'limit' not found

origin

the bug originates from the limit active binding of the Ambiorix class:

limit <- function(value) {
  if (missing(value)) {
    return(private$.limit)
  }

  private$.limit <- as.integer(limit)
}

fix

instead of private$.limit <- as.integer(limit) we should have private$.limit <- as.integer(value)

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 a pull request may close this issue.

1 participant