You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cxf.apache.org by as...@apache.org on 2014/12/23 12:56:32 UTC

cxf git commit: CXF-6175 - STS: warning message is not informative, in case if service doesn't match

Repository: cxf
Updated Branches:
  refs/heads/master a00dc60e7 -> 40c6734f9


 CXF-6175 - STS: warning message is not informative, in case if service doesn't match


Project: http://git-wip-us.apache.org/repos/asf/cxf/repo
Commit: http://git-wip-us.apache.org/repos/asf/cxf/commit/40c6734f
Tree: http://git-wip-us.apache.org/repos/asf/cxf/tree/40c6734f
Diff: http://git-wip-us.apache.org/repos/asf/cxf/diff/40c6734f

Branch: refs/heads/master
Commit: 40c6734f96ca0a3d82eb37088f43221e93cbd6f2
Parents: a00dc60
Author: Andrei Shakirin <an...@gmail.com>
Authored: Tue Dec 23 12:55:38 2014 +0100
Committer: Andrei Shakirin <an...@gmail.com>
Committed: Tue Dec 23 12:55:38 2014 +0100

----------------------------------------------------------------------
 .../org/apache/cxf/sts/operation/AbstractOperation.java     | 9 +++++----
 .../org/apache/cxf/sts/operation/TokenIssueOperation.java   | 1 +
 2 files changed, 6 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cxf/blob/40c6734f/services/sts/sts-core/src/main/java/org/apache/cxf/sts/operation/AbstractOperation.java
----------------------------------------------------------------------
diff --git a/services/sts/sts-core/src/main/java/org/apache/cxf/sts/operation/AbstractOperation.java b/services/sts/sts-core/src/main/java/org/apache/cxf/sts/operation/AbstractOperation.java
index 997369f..304ea2a 100644
--- a/services/sts/sts-core/src/main/java/org/apache/cxf/sts/operation/AbstractOperation.java
+++ b/services/sts/sts-core/src/main/java/org/apache/cxf/sts/operation/AbstractOperation.java
@@ -522,10 +522,11 @@ public abstract class AbstractOperation {
                 }
             }
             if (!foundService) {
-                LOG.log(Level.WARNING, "The Service cannot match the received AppliesTo address");
-                throw new STSException(
-                    "No service corresponding to " + address + " is known", STSException.REQUEST_FAILED
-                );
+                String msg = "No service corresponding to "
+                             + address
+                             + " is known. Check 'services' property configuration in SecurityTokenServiceProvider";
+                LOG.log(Level.SEVERE, msg);
+                throw new STSException(msg, STSException.REQUEST_FAILED);
             }
         }
         

http://git-wip-us.apache.org/repos/asf/cxf/blob/40c6734f/services/sts/sts-core/src/main/java/org/apache/cxf/sts/operation/TokenIssueOperation.java
----------------------------------------------------------------------
diff --git a/services/sts/sts-core/src/main/java/org/apache/cxf/sts/operation/TokenIssueOperation.java b/services/sts/sts-core/src/main/java/org/apache/cxf/sts/operation/TokenIssueOperation.java
index d4405d5..3ca5d22 100644
--- a/services/sts/sts-core/src/main/java/org/apache/cxf/sts/operation/TokenIssueOperation.java
+++ b/services/sts/sts-core/src/main/java/org/apache/cxf/sts/operation/TokenIssueOperation.java
@@ -210,6 +210,7 @@ public class TokenIssueOperation extends AbstractOperation implements IssueOpera
             }
         
         } catch (RuntimeException ex) {
+            LOG.log(Level.SEVERE, "Cannot issue token: " + ex.getMessage(), ex);
             STSIssueFailureEvent event = new STSIssueFailureEvent(providerParameters,
                                                               System.currentTimeMillis() - start, ex);
             publishEvent(event);