Skip to content

Conversation

@tompsota
Copy link
Member

@tompsota tompsota commented Oct 31, 2025

What's new

Describe the PR shortly

PR checklist

In-app checks:

  • callbacks are received
  • dev/prod switch works
  • README.md is updated (if applicable)
  • expo plugin is updated (if applicable)
  • example app is working

Pre-release checks:

  • CHANGELOG.md is updated
  • package.json version is increased
  • release label is applied on PR

To be checked by maintainers:

  • freeRASP logs are received
  • sdkVersion property in logs is correct
  • sdkPlatform property in logs is correct

Resolved issues

list of issues resolved by this PR

@tompsota tompsota added the release Create release after merging this PR label Oct 31, 2025
listener.registerListener(reactContext)
runOnUiThread {
Talsec.start(reactContext, config)
Talsec.start(reactContext, config, TalsecMode.BACKGROUND)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We do not give an option to set a different mode?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Opted not to do so in Flutter as well

import android.os.Handler
import android.os.HandlerThread
import android.os.Looper
import android.util.Log
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unused import?


}


Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Empty lines. Is there ktlint enabled in this module?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it is not

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release Create release after merging this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants