salmon error handling issue

include/auth.php

@@ -114,7 +114,7 @@ else {
 				// new account
 				$_SESSION['register'] = 1;
 				$openid->required = array('namePerson/friendly', 'contact/email', 'namePerson');
-				$openid->optional = array('namePerson/first');
+				$openid->optional = array('namePerson/first','media/image/aspect11');
 				goaway($openid->authUrl());
 				// NOTREACHED	
 			}

include/salmon.php

@@ -182,7 +182,9 @@ EOT;
 
 	// check for success, e.g. 2xx
 
-	if(($return_code >= 200) && ($return_code < 300)) {
+	if($return_code > 299) {
+
+		logger('slapper: compliant salmon failed. Falling back to status.net hack');
 
 		// Entirely likely that their salmon implementation is
 		// non-compliant. Let's try once more, this time only signing

mod/register.php

@@ -49,7 +49,7 @@ function register_post(&$a) {
 			$openid->identity = $openid_url;
 			$openid->returnUrl = $a->get_baseurl() . '/openid'; 
 			$openid->required = array('namePerson/friendly', 'contact/email', 'namePerson');
-			$openid->optional = array('namePerson/first');
+			$openid->optional = array('namePerson/first','media/image/aspect11');
 			goaway($openid->authUrl());
 			// NOTREACHED	
 		}