file.check_perms breaks with symlinks

[johtso]

There is inconsistent use of follow_symlinks in file.check_perms

The owner of the file is first checked using state with follow_symlinks=False, and then with get_user which defaults to follow_symlinks=True.

This causes the state to fail for symlinks that have the correct ownership but point to files that have different ownership.

Looks to be caused by this change: 7d4c470#diff-7c48191b14cdf053cf954d03cc317963L2195

[basepi]

Good catch, thank you!

@terminalmage looks like you made this change, so I just wanted to double check that we weren't missing anything. I expect both should be follow_symlinks=False, but wanted to double check.

[rallytime]

@johtso - can you take a look and see if this PR fixes your issue?

[rallytime]

Since we haven't heard of any other reports regarding this issue, I am going to close it. We are certainly happy to re-open it if people are still seeing problems.