fix: [CO-1179] UserServlet to properly return XML and scriptable atta…

…chments without defanging (#530)

* fix: [CO-1179] UserServlet force content-disposition attachment when mime-part is scriptable

* chore: cleanup

* test: add API test for UserServlet

* chore: update test name

* chore: test cleanup

- limit scope of test in NativeFormatter
- improve test in UserServletTest

* chore: cleanup

* chore: improve naming of var in test

store/src/main/java/com/zimbra/cs/service/UserServlet.java

@@ -442,7 +442,6 @@ protected boolean proxyIfRemoteTargetAccount(
    * Constructs the exteral url for a mount point. This gets the link back to the correct server
    * without need for proxying it
    *
-   * @param authToken
    * @param mpt The mount point to create the url for
    * @return The url for the mountpoint/share that goes back to the original user/share/server
    * @throws ServiceException
@@ -514,8 +513,6 @@ public static String getExternalRestUrl(OperationContext octxt, Mountpoint mpt)
    * Constructs the exteral url for a mount point. This gets the link back to the correct server
    * without need for proxying it
    *
-   * @param authToken
-   * @param mpt The mount point to create the url for
    * @return The url for the mountpoint/share that goes back to the original user/share/server
    * @throws ServiceException
    */