Skip to content
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

Jump to bottom

Update solana program show --buffers to use base64 encoding #5157

Open
jacobcreech opened this issue Mar 5, 2025 · 0 comments · May be fixed by #5164
Open

Update solana program show --buffers to use base64 encoding #5157

jacobcreech opened this issue Mar 5, 2025 · 0 comments · May be fixed by #5164
Labels
community

Comments

@jacobcreech
Copy link

Problem

When using solana program show --buffers, the RPC API body has everything set as encoding: "base58" right now. Example body sent via this CLI command:

{
    "jsonrpc": "2.0",
    "id": 1,
    "method": "getProgramAccounts",
    "params": [
        "BPFLoaderUpgradeab1e11111111111111111111111",
        {
            "commitment": "confirmed",
            "dataSlice": {
                "length": 37,
                "encoding": "base58",
                "offset": 0
            },
            "encoding": "base58",
            "filters": [
                {
                    "memcmp": {
                        "offset": 0,
                        "encoding": "base58",
                        "bytes": "2UzHM" 
                    }
                },
                {
                    "memcmp": {
                        "offset": 4,
                        "encoding": "base58",
                        "bytes": "2"
                    }
                },
                {
                    "memcmp": {
                        "offset": 5,
                        "encoding": "base58",
                        "bytes": "<ADDRESS>" 
                    }
                }
            ]
        }
    ]
  }

On another note, this call is incredibly slow on devnet due to the amount of programs deployed. I don't expect this update to fix the slowness, but it'll help.

Proposed Solution

Update to not use base58.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
community
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: use base64 instead of base58 for program show --buffers sonicfromnewyoke/agave
1 participant
@jacobcreech