You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cxf.apache.org by se...@apache.org on 2012/12/17 12:57:48 UTC

svn commit: r1422884 - in /cxf/branches/2.6.x-fixes: ./ rt/rs/security/oauth-parent/oauth/src/main/java/org/apache/cxf/rs/security/oauth/filters/ rt/rs/security/oauth-parent/oauth2/src/main/java/org/apache/cxf/rs/security/oauth2/filters/

Author: sergeyb
Date: Mon Dec 17 11:57:48 2012
New Revision: 1422884

URL: http://svn.apache.org/viewvc?rev=1422884&view=rev
Log:
Merged revisions 1422882 via svnmerge from 
https://svn.apache.org/repos/asf/cxf/trunk

........
  r1422882 | sergeyb | 2012-12-17 11:54:02 +0000 (Mon, 17 Dec 2012) | 1 line
  
  If OAuth permission restricts URI and the client fails to use a valid URI - issue the warning
........

Modified:
    cxf/branches/2.6.x-fixes/   (props changed)
    cxf/branches/2.6.x-fixes/rt/rs/security/oauth-parent/oauth/src/main/java/org/apache/cxf/rs/security/oauth/filters/AbstractAuthFilter.java
    cxf/branches/2.6.x-fixes/rt/rs/security/oauth-parent/oauth2/src/main/java/org/apache/cxf/rs/security/oauth2/filters/OAuthRequestFilter.java

Propchange: cxf/branches/2.6.x-fixes/
------------------------------------------------------------------------------
  Merged /cxf/trunk:r1422882

Propchange: cxf/branches/2.6.x-fixes/
------------------------------------------------------------------------------
Binary property 'svnmerge-integrated' - no diff available.

Modified: cxf/branches/2.6.x-fixes/rt/rs/security/oauth-parent/oauth/src/main/java/org/apache/cxf/rs/security/oauth/filters/AbstractAuthFilter.java
URL: http://svn.apache.org/viewvc/cxf/branches/2.6.x-fixes/rt/rs/security/oauth-parent/oauth/src/main/java/org/apache/cxf/rs/security/oauth/filters/AbstractAuthFilter.java?rev=1422884&r1=1422883&r2=1422884&view=diff
==============================================================================
--- cxf/branches/2.6.x-fixes/rt/rs/security/oauth-parent/oauth/src/main/java/org/apache/cxf/rs/security/oauth/filters/AbstractAuthFilter.java (original)
+++ cxf/branches/2.6.x-fixes/rt/rs/security/oauth-parent/oauth/src/main/java/org/apache/cxf/rs/security/oauth/filters/AbstractAuthFilter.java Mon Dec 17 11:57:48 2012
@@ -224,8 +224,8 @@ public class AbstractAuthFilter {
             }
         }
         if (!foundValidScope) {
-            String message = "Invalid request URI";
-            LOG.fine(message);
+            String message = "Invalid request URI: " + request.getRequestURL().toString();
+            LOG.warning(message);
         }
         return foundValidScope;
     }

Modified: cxf/branches/2.6.x-fixes/rt/rs/security/oauth-parent/oauth2/src/main/java/org/apache/cxf/rs/security/oauth2/filters/OAuthRequestFilter.java
URL: http://svn.apache.org/viewvc/cxf/branches/2.6.x-fixes/rt/rs/security/oauth-parent/oauth2/src/main/java/org/apache/cxf/rs/security/oauth2/filters/OAuthRequestFilter.java?rev=1422884&r1=1422883&r2=1422884&view=diff
==============================================================================
--- cxf/branches/2.6.x-fixes/rt/rs/security/oauth-parent/oauth2/src/main/java/org/apache/cxf/rs/security/oauth2/filters/OAuthRequestFilter.java (original)
+++ cxf/branches/2.6.x-fixes/rt/rs/security/oauth-parent/oauth2/src/main/java/org/apache/cxf/rs/security/oauth2/filters/OAuthRequestFilter.java Mon Dec 17 11:57:48 2012
@@ -122,8 +122,8 @@ public class OAuthRequestFilter extends 
             }
         }
         if (!foundValidScope) {
-            String message = "Invalid request URI";
-            LOG.fine(message);
+            String message = "Invalid request URI: " + request.getRequestURL().toString();
+            LOG.warning(message);
         }
         return foundValidScope;
     }