Sfoglia il codice sorgente

Allow edition of OAuth endpoints for testing; better configuration values

David A. Velasco 12 anni fa
parent
commit
980b31de2e

+ 0 - 2
res/layout/account_setup.xml

@@ -119,7 +119,6 @@
                 android:layout_height="wrap_content"
                 android:layout_weight="1"
                 android:ems="10"
-                android:enabled="false"
                 android:text="@string/oauth2_url_endpoint_auth"
                 android:singleLine="true"
                 android:visibility="gone" >
@@ -133,7 +132,6 @@
                 android:layout_height="wrap_content"
                 android:layout_weight="1"
                 android:ems="10"
-                android:enabled="false"
 	            android:text="@string/oauth2_url_endpoint_access"
                 android:singleLine="true"
                 android:visibility="gone" >

+ 4 - 4
res/values/oauth2_configuration.xml

@@ -1,18 +1,18 @@
 <?xml version="1.0" encoding="utf-8"?>
 <resources>
     <!-- constants that must be respected by the authorization server; if changed, the app must be rebuild -->
-    <string name="oauth2_redirect_scheme">oauth-mobile-app</string>
-    <string name="oauth2_redirect_uri">oauth-mobile-app://callback</string>
+    <string name="oauth2_redirect_scheme">owncloud</string>
+    <string name="oauth2_redirect_uri">owncloud://callback</string>
     
     <!-- values that should be provided by ownCloud server -->
     <string name="oauth2_url_endpoint_auth">http://owncloud.tuxed.net/oauth/php-oauth/authorize.php</string>
     <string name="oauth2_url_endpoint_access">http://owncloud.tuxed.net/oauth/php-oauth/token.php</string>
-    <string name="oauth2_scope">grades</string>
+    <string name="oauth2_scope">owncloud</string>
     <string name="oauth2_grant_type">authorization_code</string>	<!-- the only one supported right now -->
     <string name="oauth2_response_type">code</string>				<!-- depends on oauth2_grant_type -->
     
     <!-- values that should be agreed between app and authorization server, but can be loaded without rebuilding the app -->
-    <string name="oauth2_client_id">oc-android-test</string>		<!-- preferable that client decides this -->
+    <string name="oauth2_client_id">com.owncloud.android</string>	<!-- preferable that client decides this -->
     <string name="oauth2_client_secret"></string>					<!-- preferable that client decides this -->
     
 </resources>

+ 4 - 2
src/com/owncloud/android/authentication/AuthenticatorActivity.java

@@ -344,7 +344,8 @@ public class AuthenticatorActivity extends AccountAuthenticatorActivity
                                                                 getString(R.string.oauth2_redirect_uri), // TODO check - necessary here?      
                                                                 getString(R.string.oauth2_grant_type),
                                                                 queryParameters);
-        WebdavClient client = OwnCloudClientUtils.createOwnCloudClient(Uri.parse(getString(R.string.oauth2_url_endpoint_access)), getApplicationContext());
+        //WebdavClient client = OwnCloudClientUtils.createOwnCloudClient(Uri.parse(getString(R.string.oauth2_url_endpoint_access)), getApplicationContext());
+        WebdavClient client = OwnCloudClientUtils.createOwnCloudClient(Uri.parse(mOAuthTokenEndpointText.getText().toString().trim()), getApplicationContext());
         operation.execute(client, this, mHandler);
     }
     
@@ -516,7 +517,8 @@ public class AuthenticatorActivity extends AccountAuthenticatorActivity
         updateAuthStatus();
         
         // GET AUTHORIZATION request
-        Uri uri = Uri.parse(getString(R.string.oauth2_url_endpoint_auth));
+        //Uri uri = Uri.parse(getString(R.string.oauth2_url_endpoint_auth));
+        Uri uri = Uri.parse(mOAuthAuthEndpointText.getText().toString().trim());
         Uri.Builder uriBuilder = uri.buildUpon();
         uriBuilder.appendQueryParameter(OAuth2Constants.KEY_RESPONSE_TYPE, getString(R.string.oauth2_response_type));
         uriBuilder.appendQueryParameter(OAuth2Constants.KEY_REDIRECT_URI, getString(R.string.oauth2_redirect_uri));