[PATCH] implemented hardlink option in DAV module
Maxim Dounin
mdounin at mdounin.ru
Tue Oct 16 18:34:24 UTC 2012
Hello!
On Tue, Oct 16, 2012 at 10:04:08PM +0400, Михаил Монашёв wrote:
> Здравствуйте, Maxim.
>
> >> А если потребуется поменять atime, то:
> >> - if (ngx_set_file_time(NULL, fd, ctx->mtime) != NGX_OK) {
> >> + if (ngx_set_file_time(NULL, fd, ctx->atime) != NGX_OK) {
> >>
> >> ?
> >>
> >> А то я тут посмотрел код скриптов, работающих с вебдавом, чтоб туда
> >> идейку про перл в комменты записать. А там мы оказывается atime
> >> меняем, а не mtime.
>
> > Это cut-n-paste ошибка в виндовой части кода, которую я поленился
> > проверить и недоправил. Там должно быть date, исправленный патч
> > прилагается.
>
> >> И, кстати, Игорь прав. ;-) Глядя на твой патч возникает желание
> >> плодить методы под каждый чих, что неправильно.
>
> > С одной стороны да, с другой - иногда надо хотя бы пытаться
> > предлагать какую-то альтернативу тому ужасу, что стандартизировал
> > Microsoft. Особенно с учётом того, сколько методов они сами
> > наплодили как в стандарте на webdav, так и в дополнение к нему.
> > :)
>
> Ну тогда уж предлагать полноценную альтернативу. А то чем это
> отличается от их изысканий. Конечно, TOUCH понятен, привычен и
> наверное востребован, но это частность. Этот TOUCH станет не нужен,
> если реализовать нормальную альтернативу PROPPATCH без вывихов
Есть мнение, что TOUCH, возможно, заслуживает своего собственного
метода. Именно потому, что - понятен и привычен, и местами
востребован. И что отдельно немаловажно - прост в реализации.
> сознания. Да и там много чего можно было б реализовать правильного,
> например позаимствовать из мемкешеда cas...
CAS, кстати, в HTTP/1.1 есть - заголовки If-* позволяют всякого.
Правда, в реальной жизни это не совсем работает, или даже совсем
не работает, особенно если говорить про nginx.
> И, кстати, у nginx-а есть возможность заказывать музыку, предлагая
> свои реализации альтернатив. ИМХО.
Ну вот я TOUCH нарисовал, если приживётся - хорошо, не приживётся -
значит не судьба. :)
--
Maxim Dounin
http://nginx.com/support.html
Подробная информация о списке рассылки nginx-ru