implement remove-empty-folder in this project instead of using artifactory user plugin

[zhan9san]

Hi @allburov

Currently, the 'remove-empty-folder' feature depends on the artifactory-user-plugins

Would it be possible to implement this feature in this project and remove the post request in delete?

If this feature is valuable, feel free to assign this task to me.

[allburov]

@zhan9san hi! Great idea, we can implement the second version of it that works without an external plugin.

It's not hard to add a plugin to artifactory tho...

[zhan9san]

Hi @allburov

Do we have a plan to remove this dead code?

I tried to test this feature in 7.33.9, releases-docker.jfrog.io/jfrog/artifactory-pro:latest. After the artifact was deleted, the empty folder was deleted automatically as well.

example-repo-local:a/b-3 is an empty folder.

artifactory-access.log

2022-03-02T05:37:57.634Z [53de1bddc131e037] [ACCEPTED LOGIN]   for client : admin / 10.66.40.160.
2022-03-02T05:37:58.396Z [9cb4d91621f8e354] [ACCEPTED DELETE] example-repo-local:a/b-3  for client : _system_ / .
2022-03-02T05:38:09.326Z [58281fb62248ef7d] [ACCEPTED LOGIN]   for client : jffe@000 / 127.0.0.1.

This feature was introduced in 7.8.1.

You can get details from ArtifactoryReleaseNotes-Artifactory7.8.1

RTFACT-13618 Fixed an issue whereby, the automatic cleanup process did not prune empty folders.

[allburov]

@zhan9san hi!
I think it's better to check Artifactory version and show a warning "Your Artifactory instance is higher then 7.8.1 and empty folders are deleted automatically here"

Personally I don't use Artifactory anymore but people can still use old versions

[zhan9san]

Hi @allburov

Thank you.

Let us leave this feature as before and I'll close this ticket