search

tenex / rails-assets

The solution to assets management in Rails
https://rails-assets.org
MIT License
1.63k stars 69 forks source link

Upgrade bower and node to latest versions #487

Closed hut8 closed 2 years ago

hut8 commented 2 years ago

Resolves major security issue in #485

hut8 commented 2 years ago

@joshjordan this should be good to go. Some of the tests changed because the behavior of bower changed. If you think this is bad, I can try a workaround, but it seems to me like a lot of those tests were added because there was very unexpected behavior. Now bower matches more expected behavior, so that's why I changed the tests instead of patching rails-assets to re-implement the strange bower behavior. Your thoughts are welcome. After this, I'll re provision the server with a new, supported OS.

hut8 commented 2 years ago

I changed my mind; I'm making the tests the same as the original