lift / lift/framework

Cache results of parsing URI parameters

Open
#982 6 comments 0 reactions 1 assignee View on GitHub

@Shadowfiend is already working on this.

Since May 11, 2014.

P:Normal SiteMap T:Enhancement
Dominant language
Scala
Stars
1.3k
Forks
275
PR merge metrics
No merged PRs in 30d

Description

Menu.param allows you to define a parser and an encoder for a URI parameter. The parser is a good place to retrieve an item from a database and automatically get a 404 if it is not found, but the parsing is performed multiple times per request and the result is not cached. As a workaround, you can manually thread the result through a RequestVar. It would be nice to have a caching Menu.param available in Lift out of the box.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.