32

In play framework 1, you could use in the routes file something like this (check documentation at http://www.playframework.org/documentation/1.2.5/routes#syntax)

GET     /clients/?       Clients.index

so that the route will match /api/clients and also /api/clients/

How can I achieve the same in play framework 2?

6 Answers 6

71

From SEO point of view the same link with trailing slash is other one than link without it. It is highly recommended to always use one schema (trailed or un-trailed links).

Although there are different schools which one is better the most important is to make a 301 redirect from 'wrong' URL to the correct. You can achieve it quite easy in Play with a 'Dynamic part spanning several /'.

Personally I prefer un-trailed version, maybe because implementing it in the Play is just like writing few simple lines. Add to your routes file this rule, somewhere at the beginning (keep the slash - it's important as it's NOT considered as next slash in the spanning-group, and allows to match trailed URL's easily):

GET  /*path/  controllers.Application.untrail(path: String)

then you can just make a redirect in the controller - to the param, so it will be without the slash at the end:

Java

public static Result untrail(String path) {
   return movedPermanently("/" + path);
}

Scala

def untrail(path: String) = Action { 
  MovedPermanently("/" + path)
}

Until now, all routes ending with the slash will be redirected to the un-trailed version. Easy :)

Of course it's highly recommended to use reverse router for generating correct URL's - to minimalize redundant redirects. Also if you're hardcoding the URL somewhere (ie. in some JS or in external application) it's also better to write correct ones instead converting them every time. If you're planning to publish some public API make a note in documentation, which pattern does your application prefer, so developers will be warned and (maybe) will prepare correct calls.

What's more - it most important for GET routes as they are a subject to manipulation from the client's side. While using POST, PUT, DELETE and others you don't need (or rather, you should't) to care about redirects as they can not be changed by the user and in that way you need to remember which way you choose. In case of wrong call ie. for POST, just return a 404 error - so the developer of the 3-rd part application will be obligated to use correct endings.

8
  • 1
    Thanks. And in Scala, this looks like: def untrail(path: String) = Action { MovedPermanently("/%s".format(path)) } May 19, 2013 at 3:28
  • @ChristopherMartin, thanx, I copied your sample into answer
    – biesior
    May 20, 2013 at 7:43
  • This solution worked very well in Play 2.1.2 scala. I added routes for GET, POST, PUT and DELETE and so far they seem to redirect properly. Aug 4, 2013 at 3:38
  • 1
    @biesior Is it still the only workaround to this or have they added any feature regarding this in Play 2.3.2?
    – ajay
    Jul 28, 2014 at 9:48
  • 1
    Hello @biesior, thanks for your reply. There is one corner case, which opens 'Open Redirect' issue (cwe.mitre.org/data/definitions/601.html). Just FYI: in case the path has something like /////somesite.com/ the browser will visit the other site.
    – Barys
    Dec 23, 2019 at 12:37
9

I've managed to come up with something, it wasn't as simple as I hoped, but it's no rocket science either

import play.api.mvc.RequestHeader

import play.api.Play.current

class NormalizedRequest(request: RequestHeader) extends RequestHeader {

  val headers = request.headers
  val queryString = request.queryString
  val remoteAddress = request.remoteAddress
  val method = request.method

  val path = request.path.stripSuffix("/")
  val uri = path + {
    if(request.rawQueryString == "") ""
    else "?" + request.rawQueryString
  }
}

object NormalizedRequest {
  def apply(request: RequestHeader) = new NormalizedRequest(request)
}

ans then I use it like this in Global.scala

override def onRouteRequest(request: RequestHeader): Option[Handler] = {
  super.onRouteRequest(NormalizedRequest(request))
}
2
  • 2
    Examples of Global interception in practice are few and far between, +1 for a real world case Nov 3, 2012 at 13:56
  • Is there an example of the equivalent code in Java?
    – Devis L.
    Jun 11, 2018 at 17:28
6

Updated the example by @opensas and @lloydmeta for play 2.5

/**
  * HttpRequestHandler that removes trailing slashes from requests.
  */
class TrailingSlashNormaliserHttpRequestHandler(router: Router, errorHandler: HttpErrorHandler, configuration: HttpConfiguration, filters: HttpFilters) extends HttpRequestHandler {

  private val default = new DefaultHttpRequestHandler(router, errorHandler, configuration, filters)

  override def handlerForRequest(request: RequestHeader): (RequestHeader, Handler) = {
    default.handlerForRequest(removeTrailingSlash(request))
  }

  private def removeTrailingSlash(origReq: RequestHeader): RequestHeader = {
    if (origReq.path.endsWith("/") && origReq.path != "/") {
      val path = origReq.path.stripSuffix("/")
      if (origReq.rawQueryString.isEmpty) {
        origReq.copy(path = path, uri = path)
      }else {
        origReq.copy(path = path, uri = path + s"?${origReq.rawQueryString}")
      }
    } else {
      origReq
    }
  }
}

see https://www.playframework.com/documentation/2.5.x/ScalaHttpRequestHandlers for instructions on how to apply the handler

2

An update to the other answers here, for Play 2.8:

import play.api.OptionalDevContext
import play.api.http._
import play.api.mvc.{Handler, RequestHeader}
import play.api.routing.Router
import play.core.WebCommands

import javax.inject.Inject

class TrailingSlashHandler @Inject() (
  webCommands: WebCommands,
  optDevContext: OptionalDevContext,
  router: Router,
  errorHandler: HttpErrorHandler,
  configuration: HttpConfiguration,
  filters: HttpFilters
) extends DefaultHttpRequestHandler(
    webCommands,
    optDevContext,
    router,
    errorHandler,
    configuration,
    filters
  ) {

  override def handlerForRequest(request: RequestHeader): (RequestHeader, Handler) =
    super.handlerForRequest(removeTrailingSlash(request))

  private def removeTrailingSlash(originalRequest: RequestHeader): RequestHeader =
    if (originalRequest.path.endsWith("/") && originalRequest.path != "/") {
      val normalizedPath = originalRequest.path.stripSuffix("/")
      val normalizedUri =
        if (originalRequest.queryString.isEmpty) originalRequest.target.uriString.stripSuffix("/")
        else originalRequest.target.uriString.replaceFirst("/\\?", "?")
      originalRequest.withTarget(
        originalRequest.target
          .withPath(normalizedPath)
          .withUriString(normalizedUri)
      )
    } else {
      originalRequest
    }
}

And then just register via placing this in your conf: play.http.requestHandler = your.package.here.TrailingSlashHandler

1

This is based on opensas's answer, just simplified a bit to reuse Play's built-in copy method on RequestHeader so that all the things in the original RequestHeader are kept, like id, tags, version, secure, etc.

import play.api.GlobalSettings
import play.api.mvc.{Handler, RequestHeader}

trait TrailingSlashNormaliser extends GlobalSettings {

  def removeTrailingSlash(origReq: RequestHeader): RequestHeader = {
    if (origReq.path.endsWith("/")) {
      val path = origReq.path.stripSuffix("/")
      if (origReq.rawQueryString.isEmpty)
        origReq.copy(path = path, uri = path)
      else
        origReq.copy(path = path, uri = path + s"?${origReq.rawQueryString}")
    } else {
      origReq
    }
  }

  override def onRouteRequest(request: RequestHeader): Option[Handler] = 
    super.onRouteRequest(removeTrailingSlash(request))

}

/**
 * Global object that removes trailing slashes from requests.
 */
object Global extends TrailingSlashNormaliser
1
  • 1
    if (origReq.path.endsWith("/") && origReq.path != "/") {
    – user56250
    Oct 27, 2015 at 17:15
0

Add the entry twice in your route file. One with the slash and one without.

3
  • 1
    well, yes, that's the workaround I've found so far, I'm just trying to keep my routes clean
    – opensas
    Nov 2, 2012 at 5:55
  • 1
    fun, double the routes, build times are bad enough as is, NO THANKS ;-) Jan 4, 2013 at 15:36
  • Thats a bad practice. Do not double anything in your code never. Choose just one schema and use that
    – nicowernli
    Aug 5, 2013 at 20:07

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Not the answer you're looking for? Browse other questions tagged or ask your own question.