Skip to content

Fix balance check and trading transaction 404 errors #9

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Oct 12, 2025
Merged

Fix balance check and trading transaction 404 errors #9

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
73e7f92
Initial plan
Copilot Oct 12, 2025
dc8addc
Fix endpoint URLs for balance and trading operations
Copilot Oct 12, 2025
01ec837
Add documentation for endpoint fix
Copilot Oct 12, 2025
707b9ef
Clarify affected methods in documentation
Copilot Oct 12, 2025
115590b
Improve documentation clarity based on code review feedback
Copilot Oct 12, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
57 changes: 57 additions & 0 deletions ENDPOINT_FIX_SUMMARY.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,57 @@
# Endpoint Fix Summary

## Issue
Users were experiencing 404 errors when calling:
- `client.GetBalance()` - Balance check method
- `client.buy_token()` - Buy transaction method
- Other trading-related endpoints

Error messages:
```
{'success': False, 'error': 'Failed to get buy transaction: 404 - <html><body><h1>File Not Found</h1><hr><i>uWebSockets/20 Server</i></body></html>'}
```

## Root Cause
The SDK was using `https://api6.axiom.trade` as the base URL for balance and trading endpoints, but the actual Axiom Trade application uses `https://axiom.trade/api`.

## Fix Applied
Changed `BASE_URL_API` in `axiomtradeapi/content/endpoints.py`:
- **Before:** `BASE_URL_API = "https://api6.axiom.trade"`
- **After:** `BASE_URL_API = "https://axiom.trade/api"`

## Impact
This fix affects the following endpoints:

### Balance Endpoints
- `/sol-balance` → `https://axiom.trade/api/sol-balance`
- `/batched-sol-balance` → `https://axiom.trade/api/batched-sol-balance`
- `/token-balance` → `https://axiom.trade/api/token-balance`

### Trading Endpoints
- `/buy` → `https://axiom.trade/api/buy`
- `/sell` → `https://axiom.trade/api/sell`
- `/send-transaction` → `https://axiom.trade/api/send-transaction`

## Affected Methods

### Default Client (`AxiomTradeClient`)
These methods in the default `AxiomTradeClient` class are now fixed:
- `GetBalance()` - Get balance for a single wallet
- `GetBatchedBalance()` - Get balances for multiple wallets
- `buy_token()` - Buy tokens
- `sell_token()` - Sell tokens
- `get_token_balance()` - Get token balance

### Enhanced Client (`EnhancedAxiomTradeClient`)
These methods in the enhanced client are now fixed:
- `get_sol_balance()` - Get SOL balance
- `get_token_balance()` - Get token balance

## Verification
All endpoint URLs have been verified to match the expected format used by the Axiom Trade web application.

## Notes
- Authentication endpoints (like `refresh_access_token` at `api9.axiom.trade`) continue to use their respective base URLs - these were already working correctly
- Trending tokens endpoint (`/meme-trending` at `api6.axiom.trade`) remains unchanged - this was already working correctly
- Portfolio endpoint (`/portfolio` at `api6.axiom.trade`) remains unchanged - this was already working correctly
- **Only balance and trading endpoints** were affected by this fix, as they are the ones that should use `https://axiom.trade/api`
2 changes: 1 addition & 1 deletion axiomtradeapi/content/endpoints.py
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
class Endpoints:
# Base Urls
BASE_URL_API = "https://api6.axiom.trade"
BASE_URL_API = "https://axiom.trade/api"
BASE_URL = "https://axiom.trade"

# Endpoints
Expand Down