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

--fix Vul (time/chrono/sqlx/libsqlite3-sys) #271

Closed
wants to merge 2 commits into from
Closed

--fix Vul (time/chrono/sqlx/libsqlite3-sys) #271

wants to merge 2 commits into from

Conversation

Rusty-Weasel
Copy link

Cargo audit does not mention any vul's now. 😃

@rustdesk
Copy link
Owner

rustdesk commented Jul 3, 2023

👍

@rustdesk
Copy link
Owner

rustdesk commented Jul 3, 2023

Could you please give a link about this vul?

@Rusty-Weasel
Copy link
Author

@Rusty-Weasel
Copy link
Author

I started also working on Client Version, therefore i like to know whats best way to handle further:

First cargo audit run:
image

Left side before starting, right side result after many changes, now it's clean and still build on linux (upper right corner)
image

needed to clone the following repros local and changed parts:
image

Would it be a good way to fork all these repros an then put the changes into fork an send pr to each project,
then i can switch my fork to point to the forked deb parts and could try the github action build way, to see if other system throw any errors.

or do you have a better idea how to handle?

@rustdesk
Copy link
Owner

rustdesk commented Jul 3, 2023

It is a horrible job.

@rustdesk
Copy link
Owner

rustdesk commented Jul 3, 2023

or do you have a better idea how to handle?

I have no idea either, so I dare not, even have no courage to run cargo clippy on RustDesk. :(

@Rusty-Weasel
Copy link
Author

i've done it for u 😄 , mostly easy things as i saw... only scrab lib will got work i think 😄

@Rusty-Weasel
Copy link
Author

First parts done, repros forked and changed everthing, local build on forks work,
now checking out this github action part

@dinger1986
Copy link
Contributor

did it test ok?

@Rusty-Weasel Rusty-Weasel closed this by deleting the head repository Aug 2, 2023
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.

3 participants