Skip to content

Commit 3516be5

Browse files
authored
chore: Add unit tests for ToolboxSyncClient (#225)
* chore: Add unit test cases * chore: Delint * feat: Warn on insecure tool invocation with authentication This change introduces a warning that is displayed immediately before a tool invocation if: 1. The invocation includes an authentication header. 2. The connection is being made over non-secure HTTP. > [!IMPORTANT] The purpose of this warning is to alert the user to the security risk of sending credentials over an unencrypted channel and to encourage the use of HTTPS. * fix!: Warn about https only during tool initialization * chore: Add unit tests for ToolboxSyncClient * chore: Delint * chore: Refactor sync client tests to be grouped together in classes These also help identify differences with the existing async client tests. * chore: Delint
1 parent a4f0f25 commit 3516be5

File tree

2 files changed

+596
-0
lines changed

2 files changed

+596
-0
lines changed

packages/toolbox-core/src/toolbox_core/client.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,8 @@
1111
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
1212
# See the License for the specific language governing permissions and
1313
# limitations under the License.
14+
15+
1416
import types
1517
from typing import Any, Callable, Coroutine, Mapping, Optional, Union
1618

0 commit comments

Comments
 (0)