IOS Register method ::: Argument 7 (double) of RNAppAuth.register must not be null

[sampertelus]

Issue

config

{
  "additionalParameters": {}, 
  "clientId": "interactive.public", 
  "connectionTimeoutSeconds": 5, 
  "issuer": "https://demo.duendesoftware.com", 
  "redirectUrls": ["com.myapp.dev.auth:/oauthredirect"]
}

Method:

const handleRegister = useCallback(async provider => {
    try {
      const config = configs[provider];
      console.log(config)
      const newAuthState = await register(config);

      setAuthState({
        hasLoggedInOnce: true,
        provider,
        ...newAuthState,
      });
    } catch (error) {
      Alert.alert('Failed to log in', error.message);
    }
  }, []);

After pressing button It will show this error:

Xcode log:

2022-12-02 12:27:39.292193-0500 Myapp[13279:112264] [javascript] { issuer: 'https://demo.duendesoftware.com',
  clientId: 'interactive.public',
  redirectUrls: [ [Getter/Setter] ],
  additionalParameters: {},
  connectionTimeoutSeconds: 5 }
2022-12-02 12:27:39.292977-0500 Myapp[13279:112080] [native] Argument 7 (double) of RNAppAuth.register must not be null
2022-12-02 12:27:39.294845-0500 Myapp[13279:112080] [native] Argument 7 (<null>) of RNAppAuth.register could not be processed. Aborting method call.

---

Environment

• Your Identity Provider: demo.duendesoftware.com
• Platform that you're experiencing the issue on: IOS
• Are you using Expo? NO

[sampertelus]

The fix is to change index.js line 171

const nativeMethodArguments = [
    issuer,
    redirectUrls,
    responseTypes,
    grantTypes,
    subjectType,
    tokenEndpointAuthMethod,
    additionalParameters,
    convertTimeoutForPlatform(Platform.OS, connectionTimeoutSeconds),
    serviceConfiguration,
  ];

The order of paramter does not match with IOS method

[sampertelus]

After the fix it show error
Access Token not valid

[tudormarze]

Opened a PR here. I think it's best to fix in iOS, otherwise changing only in JS will break the order in Android.