pando85
These changes were made to extend SD card life. It has a con because you will lose you logs between reboots but for me it is ok. Also, I moved...
It could be related with server encoding, I don't remember right now and I can't tested, sorry.
here: https://github.com/pando85/crow I mean, it is difficult now for me because there are a lot of changes there (even the name was changed to be consistent) while I thought this...
I will add this when I decide how to make in rash-sh/rash#39.
After some research I could say that the more appropriate solution could be similar to [this](https://github.com/dani-garcia/bitwarden_rs/issues/600). Setting up a configurable header for origin IP address and trying to pick IP...
After more research... I found that Nginx has the most professional solution in [ngx_http_realip_module](https://nginx.org/en/docs/http/ngx_http_realip_module.html). But for Kanidm this could be over complicated... I propose to just make [actix_web log format](https://docs.rs/actix-web/0.5.8/actix_web/middleware/struct.Logger.html)...
Ok, we could have an audit problem already. I continue investigating about this and by default actix-web log [this](https://github.com/actix/actix-web/blob/d5ceae20740837bddef85053c3354452b437877a/src/info.rs#L158) as remote IP. Basically by default we are logging as remote...
Yes, I did a PR to actix-web to remove that behaviour and respond with the IP by default. We must update (to 3.0) or backport (to 2.X) and fix this....
In my opinion, going to environment variable value/path and/or configuration files could be great but it's not a priority. If we add this must be and aggregation to merge with...
I'm totally accord with that. We have to focus in converge in those principles.