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

[SD WebUI Forge] Output Image Broken when using Output Sharing (txt2img/img2img) #500

Closed
Targren opened this issue Mar 4, 2024 · 2 comments · Fixed by #512
Closed

[SD WebUI Forge] Output Image Broken when using Output Sharing (txt2img/img2img) #500

Targren opened this issue Mar 4, 2024 · 2 comments · Fixed by #512
Labels
bug Something isn't working

Comments

@Targren
Copy link
Contributor

Targren commented Mar 4, 2024

StabilityMatrix Version: 2.9.0

Forge version: f0.0.17v1.8.0rc-latest-273-gb9705c58

Launch Options:
--always-gpu --listen --enable-insecure-extension-access --theme dark --pin-shared-memory --cuda-malloc --cuda-stream

With "output sharing" option enabled, interstitial "in progress" images appear correctly, but when generation completes, it appears as a "broken image." The gen is saved correctly in the output directory, and shows up correctly if I install an image browser extension (the problem persists with the extension removed).

If I turn off output sharing, results display as expected.
Capture

@mohnjiles mohnjiles added the bug Something isn't working label Mar 4, 2024
@mohnjiles
Copy link
Contributor

Seems to be related to the path traversal exploit fixes in Gradio, for now, could you try adding --gradio-allowed-path G:\SMData\Data\Images to the launch args as a workaround? Where the path is your path to the shared output folder.

@Targren
Copy link
Contributor Author

Targren commented Mar 4, 2024

Looks like that did it, thanks. Took me a couple of tries (putting a trailing slash on it made it not work, in case anyone else reading this has the same problem)

@mohnjiles mohnjiles mentioned this issue Mar 12, 2024
allwavemedia pushed a commit to allwavemedia/StabilityMatrix that referenced this issue Mar 13, 2024
Add Tkinter prerequisite and handle it
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants