Sanchaga/pr 37 reviewbottest

[sanchaga-splunk]

Features

List any features you're adding to this app (e.g. new actions, parameters, auth methods, etc.)

Bug Fixes

Describe any bugs you've fixed in this app, and how they were fixed

Manual Documentation

Have you made any changes that should be documented in manual_readme_content.md?

The following changes require documentation in manual_readme_content.md:

• New, updated, or removed REST handlers
• New, updated, or removed authentication methods, especially complex methods like OAuth
• Compatibility considerations with respect to deployment types (e.g. actions that cannot be run on cloud or an automation broker)

• I have verified that manual documentation has been updated where appropriate

Other information

---

Please refer to our Contribution Guide for any questions on submitting a pull request.

Thanks for contributing!

[sanchaga-splunk]

Issue: _init_session() is called from initialize(), which is invoked before every action including test_connectivity.

Suggested fix: In the test_connectivity handler, after _init_session succeeds with OAuth, make at least one authenticated API call (e.g.

[sanchaga-splunk]

Issue: self._use_oauth = self._oauth_token_url is not None and self._oauth_client_id is not None and self._oauth_client_secret is not None — if only one or two OAuth fields are set, _use_oauth is False and the connector silently falls back to API-key auth.

Suggested fix: In initialize(), after computing _use_oauth, add an explicit check: if not _use_oauth and any of (self._username, self._password, self._api_key) is None, call set_status(APP_ERROR, 'No complete authentication method configured.