Skip to content

Commit

Permalink
Merge pull request #258 from robfletcher/master
Browse files Browse the repository at this point in the history
SPIN-1883 debug why redirect back from meechum gets rejected
  • Loading branch information
robfletcher authored Jul 12, 2016
2 parents 1287e81 + 1ecfb45 commit a5d3fef
Showing 1 changed file with 6 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -16,10 +16,11 @@

package com.netflix.spinnaker.gate.controllers

import javax.servlet.http.HttpServletResponse
import groovy.util.logging.Slf4j
import com.netflix.spinnaker.gate.security.SpinnakerUser
import com.netflix.spinnaker.gate.security.rolesprovider.UserRolesSyncer
import com.netflix.spinnaker.security.User
import groovy.util.logging.Slf4j
import org.apache.commons.lang.exception.ExceptionUtils
import org.springframework.beans.factory.annotation.Autowired
import org.springframework.beans.factory.annotation.Value
Expand All @@ -28,8 +29,6 @@ import org.springframework.web.bind.annotation.RequestMethod
import org.springframework.web.bind.annotation.RequestParam
import org.springframework.web.bind.annotation.RestController

import javax.servlet.http.HttpServletResponse

@Slf4j
@RestController
@RequestMapping("/auth")
Expand All @@ -43,7 +42,7 @@ class AuthController {
"Louis, I think this is a start of a beautiful friendship!",
"Roads? Where we're going we don't need roads!",
"Say hello to my little friend!",
"I wish we could chat longer, but I'm having and old friend for dinner. Bye!",
"I wish we could chat longer, but I'm having an old friend for dinner. Bye!",
"Hodor.",
]

Expand Down Expand Up @@ -90,6 +89,9 @@ class AuthController {
return false
}

log.info "validDeckRedirect($to)..."
log.info " toURL=[host:$toURL.host, port:$toURL.port]"
log.info "deckBaseUrl=[host:$deckBaseUrl.host, port:$deckBaseUrl.port]"
return toURL.host == deckBaseUrl.host &&
toURL.port == deckBaseUrl.port
}
Expand Down

0 comments on commit a5d3fef

Please sign in to comment.