odnoklassniki / ok-android-sdk

Android SDK & samples for native apps integrated with OK.RU
Apache License 2.0
20 stars 28 forks source link

Not-null assertion в Odnoklassniki.request() #43

Closed sntgl closed 1 year ago

sntgl commented 1 year ago

Сейчас в методе request класса Odnoklassniki используется not-null assertion для поля mAccessToken.

У нас происходили гонки состояний, поэтому приложение могло падать на этом assert'e (порядка 150 падений в день).

Предлагается сделать этот вызов безопасным:

        mAccessToken?.let { accessToken ->
            if (mode.contains(OkRequestMode.SIGNED)) {
                signParameters(requestParams)
                requestParams[PARAM_ACCESS_TOKEN] = accessToken
            }
        }
lintax commented 1 year ago

PR вмержен