all of this is OK with me :)
Just wanted to make sure our ideas of the release are aligned more or less. I would feel comfortable to have (2) less strict in the way you propose. For the release candidates - I thought we agreed on to have - we can use this stricter rule.
Agreed?
Oliver
Just wanted to play "big boss" and make
Ingo Brunberg wrote:
Now that you ask, I must admit that point (2) of your proposal sounds very strict (just like Linus Torvalds before releasing kernel 2.6.0).
I'd rather like to see something like this appended: Non critical enhancements (or new features) that don't break backwards API compatibility are allowed.
The term "non critical" may have another meaning on the client side than on the server side, where the possibility of breaking something is far greater.
In my opinion your strict number (2) applies when we reach the Release Candidate (-rc) phase (we're currently in beta). Or do you plan to directly release a 2.0 final after a few beta versions?
In any case, feel free to disagree. I really won't be too angry with you.
Ingo
Just asking, not griping, isn't this a new feature or at least enhancement? Are you sure this should go into the release branch?
Does it fit into the proposal "What goes into the release branch" I made:
(1) The aim of the release branch is to bring the release to a final state as fast as possible
(2) In no case do new features go into the release branch
(3) No archtiectural changes go into the release branch
(4) Bugs are to be fixed as directly as possible without any general
changes and with the least amount of changes
If not do you disagree with the proposal?
Cheers :)
Oliver
[EMAIL PROTECTED] wrote:
ib 2004/02/25 08:33:21
Modified: webdavclient/clientlib/src/java/org/apache/webdav/lib Tag:
SLIDE_2_0_RELEASE_BRANCH WebdavSession.java
WebdavResource.java
Log:
Allow to provide custom credentials for authentication. This will most
likely be used when NTLM authentication is required.
Revision Changes Path
No revision
No revision
1.2.2.2 +28 -7 jakarta-slide/webdavclient/clientlib/src/java/org/apache/webdav/lib/WebdavSession.java
Index: WebdavSession.java
===================================================================
RCS file: /home/cvs/jakarta-slide/webdavclient/clientlib/src/java/org/apache/webdav/lib/WebdavSession.java,v
retrieving revision 1.2.2.1
retrieving revision 1.2.2.2
diff -u -r1.2.2.1 -r1.2.2.2
--- WebdavSession.java 5 Feb 2004 15:51:21 -0000 1.2.2.1
+++ WebdavSession.java 25 Feb 2004 16:33:21 -0000 1.2.2.2
@@ -73,6 +73,11 @@
protected HttpClient client;
/**
+ * Credentials to use for authentication
+ */
+ protected Credentials hostCredentials = null;
+
+ /**
* The hostname to use for the proxy, if any
*/
protected String proxyHost = null;
@@ -146,12 +151,19 @@
if (proxyHost != null && proxyPort > 0)
hostConfig.setProxy(proxyHost, proxyPort);
- String userName = httpURL.getUser();
- if (userName != null && userName.length() > 0) {
- String password = httpURL.getPassword();
+ if (hostCredentials == null) {
+ String userName = httpURL.getUser();
+ if (userName != null && userName.length() > 0) {
+ hostCredentials =
+ new UsernamePasswordCredentials(userName,
+ httpURL.getPassword());
+ }
+ }
+
+ if (hostCredentials != null) {
HttpState clientState = client.getState();
clientState.setCredentials(null, httpURL.getHost(),
- new UsernamePasswordCredentials(userName, password));
+ hostCredentials);
clientState.setAuthenticationPreemptive(true);
}
@@ -162,6 +174,15 @@
}
return client;
+ }
+
+ /**
+ * Set credentials for authentication.
+ *
+ * @param credentials The credentials to use for authentication.
+ */
+ public void setCredentials(Credentials credentials) {
+ hostCredentials = credentials;
}
/** Set proxy info, to use proxying.
1.3.2.3 +41 -3 jakarta-slide/webdavclient/clientlib/src/java/org/apache/webdav/lib/WebdavResource.java
Index: WebdavResource.java
===================================================================
RCS file: /home/cvs/jakarta-slide/webdavclient/clientlib/src/java/org/apache/webdav/lib/WebdavResource.java,v
retrieving revision 1.3.2.2
retrieving revision 1.3.2.3
diff -u -r1.3.2.2 -r1.3.2.3
--- WebdavResource.java 5 Feb 2004 15:51:21 -0000 1.3.2.2
+++ WebdavResource.java 25 Feb 2004 16:33:21 -0000 1.3.2.3
@@ -210,6 +210,26 @@
* The constructor.
*
* @param httpURL The specified http URL.
+ * @param credentials The credentials to use for authentication.
+ * @param action The action to set properties of this resource.
+ * @param depth The depth to find properties.
+ * @exception HttpException
+ * @exception IOException
+ * @see #setDefaultAction(int)
+ */
+ public WebdavResource(HttpURL httpURL, Credentials credentials, int action,
+ int depth)
+ throws HttpException, IOException {
+
+ setCredentials(credentials);
+ setHttpURL(httpURL, action, depth);
+ }
+
+
+ /**
+ * The constructor.
+ *
+ * @param httpURL The specified http URL.
* @param action The action to set properties of this resource.
* @param depth The depth to find properties.
* @exception HttpException
@@ -306,6 +326,24 @@
public WebdavResource(String escapedHttpURL)
throws HttpException, IOException {
+ setHttpURL(escapedHttpURL);
+ }
+
+
+ /**
+ * The constructor.
+ * It must be put an escaped http URL as an argument.
+ *
+ * @param escapedHttpURL The escaped http URL string.
+ * @param credentials The credentials used for Authentication.
+ * @exception HttpException
+ * @exception IOException
+ * @see #setDefaultAction(int)
+ */
+ public WebdavResource(String escapedHttpURL, Credentials credentials)
+ throws HttpException, IOException {
+
+ setCredentials(credentials);
setHttpURL(escapedHttpURL);
}
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
.
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
