Fixes AEP FIM SQL template variable injection bug

NOAA-OWP / hydrovis

Other

12 stars 10 forks source link

Fixes AEP FIM SQL template variable injection bug #964

Closed shawncrawley closed 1 week ago

shawncrawley commented 1 week ago

A line was committed in PR #949 that was supposed to replace {fim_version} in an sql string with the actual FIM_VERSION. The replace was treated as an in-place operation, rather than re-assigning the replace output back to the sql string... Rookie mistake.

shawncrawley commented 1 week ago

I manually put this change in place in TI so we can move forward with running AEP FIM in the meantime prior to this getting merged and deployed officially.