Skip to content

feat(variables): Overlay-Resources folder variable. (#2988)#3007

Merged
ebiggz merged 1 commit intocrowbartools:v5from
CKY-:v5-overlay-rescoreses-path-variable
Feb 20, 2025
Merged

feat(variables): Overlay-Resources folder variable. (#2988)#3007
ebiggz merged 1 commit intocrowbartools:v5from
CKY-:v5-overlay-rescoreses-path-variable

Conversation

@CKY-
Copy link
Collaborator

@CKY- CKY- commented Feb 19, 2025

Description of the Change

add a variable to get the system path of the overlay-resources path
allow traversal of array of paths to be used in play video, play sound, show image and obs;s path effects.

Applicable Issues

#2988

Testing

$overlayResourcesPath/audio/ding.wav
$overlayResourcesPath[audio, ding.wav]
$overlayResourcesPath[audio/ding.wav]
slash oriantiation dont matter on windows but could be affected on other systems
both return

"C:\Users\cky\AppData\Roaming\Firebot\v5\overlay-resources\audio/ding.wav"

path.resolve is um dumb and allows you to go where you should not
$overlayResourcesPath[/etc/passwd]
n

Screenshots

@CKY- CKY- requested a review from ebiggz as a code owner February 19, 2025 21:58
Copy link
Member

@ebiggz ebiggz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Epic, thanks for this!

@ebiggz ebiggz merged commit 65f4b3e into crowbartools:v5 Feb 20, 2025
1 check passed
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

Comments