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.

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

Increase Resolution and Direct to Meshery cloud page #14

Merged
merged 6 commits into from
Jun 1, 2023

Conversation

Abhishek-kumar09
Copy link
Contributor

No description provided.

@github-actions
Copy link

github-actions bot commented Jun 1, 2023

@Abhishek-kumar09
Copy link
Contributor Author

Abhishek-kumar09 commented Jun 1, 2023

Screenshot 2023-06-02 at 2 39 27 AM
When resolution was 1920*1080
Getting argument list too long with high resolution picture, because the image is converted to base64 and given via shell, the shell has limitation of character accepting.

The file approach, which was the original approach didn't worked out due to our https server issues (was fine with HTTP).

Anyways, another approach could be a node environment to do the same thing, the program wont have that limitation.

Signed-off-by: Abhishek kr <[email protected]>
@Abhishek-kumar09 Abhishek-kumar09 merged commit 6687523 into master Jun 1, 2023
@leecalcote
Copy link
Contributor

The "file approach" - I'm baffled as to why SSL would cause an issue. Was the "file approach" using another GitHub Action?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants