[java-identity-provider COMMIT] in /trunk: idp-profile-api/src/main/java/net/shibboleth/idp/profile/AbstractProfileAc...

[noreply at shibboleth.net]

Author: lajoie
Date: Fri Jul 20 10:52:28 2012
New Revision: 4213

URL: http://svn.shibboleth.net/view/java-identity-provider?rev=4213&view=rev
Log:
Rework some Event IDs to make them a bit broader

Modified:
    trunk/idp-profile-api/src/main/java/net/shibboleth/idp/profile/AbstractProfileAction.java
    trunk/idp-profile-api/src/main/java/net/shibboleth/idp/profile/EventIds.java
    trunk/idp-profile-api/src/main/java/net/shibboleth/idp/profile/InvalidInboundMessageException.java
    trunk/idp-profile-impl/src/main/java/net/shibboleth/idp/profile/impl/FilterAttributes.java
    trunk/idp-profile-impl/src/main/java/net/shibboleth/idp/profile/impl/ResolveAttributes.java
    trunk/idp-profile-impl/src/main/java/net/shibboleth/idp/profile/impl/SelectRelyingPartyConfiguration.java
    trunk/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/impl/profile/AddSamlMetadataToMessageContext.java
    trunk/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/impl/profile/SelectProfileConfiguration.java
    trunk/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/impl/profile/saml1/AddAttributeStatementToAssertion.java
    trunk/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/impl/profile/saml1/AddAudienceRestrictionToAssertions.java
    trunk/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/impl/profile/saml1/AddNotOnOrAfterConditionToAssertions.java
    trunk/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/impl/profile/saml1/AddResponseShell.java
    trunk/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/impl/profile/saml2/AddAttributeStatementToAssertion.java
    trunk/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/impl/profile/saml2/AddAudienceRestrictionToAssertions.java
    trunk/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/impl/profile/saml2/AddNotOnOrAfterConditionToAssertions.java
    trunk/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/impl/profile/saml2/AddResponseShell.java
    trunk/idp-saml-impl/src/test/java/net/shibboleth/idp/saml/impl/profile/saml1/AddAttributeStatementToAssertionTest.java

Modified: trunk/idp-profile-api/src/main/java/net/shibboleth/idp/profile/AbstractProfileAction.java
URL: http://svn.shibboleth.net/view/java-identity-provider/trunk/idp-profile-api/src/main/java/net/shibboleth/idp/profile/AbstractProfileAction.java?rev=4213&r1=4212&r2=4213&view=diff
==============================================================================
--- trunk/idp-profile-api/src/main/java/net/shibboleth/idp/profile/AbstractProfileAction.java (original)
+++ trunk/idp-profile-api/src/main/java/net/shibboleth/idp/profile/AbstractProfileAction.java Fri Jul 20 10:52:28 2012
@@ -177,7 +177,7 @@
                 profileContextLookupStrategy.apply(springRequestContext);
         if (profileRequestContext == null) {
             log.error("Action {}: IdP profile request context is not available", getId());
-            return ActionSupport.buildEvent(this, EventIds.NO_PROFILE_CTX);
+            return ActionSupport.buildEvent(this, EventIds.INVALID_PROFILE_CTX);
         }
 
         profileRequestContext.setHttpRequest(httpRequest);

Modified: trunk/idp-profile-api/src/main/java/net/shibboleth/idp/profile/EventIds.java
URL: http://svn.shibboleth.net/view/java-identity-provider/trunk/idp-profile-api/src/main/java/net/shibboleth/idp/profile/EventIds.java?rev=4213&r1=4212&r2=4213&view=diff
==============================================================================
--- trunk/idp-profile-api/src/main/java/net/shibboleth/idp/profile/EventIds.java (original)
+++ trunk/idp-profile-api/src/main/java/net/shibboleth/idp/profile/EventIds.java Fri Jul 20 10:52:28 2012
@@ -20,44 +20,50 @@
 /** Constants to use for {@link org.springframework.webflow.execution.Event} IDs. */
 public final class EventIds {
 
-    /** Incoming request did not contains a {@link ProfileRequestContext}. */
-    public static final String NO_PROFILE_CTX = "NoProfileCtx";
+    /**
+     * ID of event returned if the {@link ProfileRequestContext} associated with the current request is missing or
+     * corrupt in some way.
+     */
+    public static final String INVALID_PROFILE_CTX = "InvalidProfileContext";
 
     /**
-     * ID of the event returned if an expected {@link org.opensaml.messaging.context.MessageContext} is missing or
-     * invalid in some way.
+     * ID of the event returned if a {@link org.opensaml.messaging.context.MessageContext} is missing or corrupt in some
+     * way.
      */
     public static final String INVALID_MSG_CTX = "InvalidMessageContext";
 
     /**
-     * ID of the event returned if a {@link org.opensaml.messaging.context.MessageContext} does not contain a necessary
-     * {@link org.opensaml.messaging.context.BasicMessageMetadataContext}.
+     * ID of the event returned if a {@link org.opensaml.messaging.context.BasicMessageMetadataContext} is missing or
+     * corrupt in some way.
      */
-    public static final String NO_MSG_MD = "NoMessageMetadata";
+    public static final String INVALID_MSG_MD = "InvalidMessageMetadata";
 
     /**

[... 449 lines stripped ...]