Re: [PR] Decouple offline hash strength from online [couchdb]

2023-11-07 Thread via GitHub
big-r81 commented on PR #4814: URL: https://github.com/apache/couchdb/pull/4814#issuecomment-1800865306 Solved the "OpenSSL not found" error. Installed the wrong architecture (x86 instead x64). Needed to tweak the 'rebar.config' again, that the liibrary linking is working:

Re: [PR] Decouple offline hash strength from online [couchdb]

2023-11-07 Thread via GitHub
big-r81 commented on PR #4814: URL: https://github.com/apache/couchdb/pull/4814#issuecomment-1800788556 First compile run: ``` ==> fast_pbkdf2 (compile) Compiling c:/Users/couchdb/Documents/couchdb/src/fast_pbkdf2/c_src/fast_pbkdf2.c ERROR: compile failed while processing

Re: [PR] Snap updated to 3.3.2 [couchdb-pkg]

2023-11-07 Thread via GitHub
sklassen commented on code in PR #119: URL: https://github.com/apache/couchdb-pkg/pull/119#discussion_r1385878211 ## README-SNAP.md: ## @@ -251,30 +277,30 @@ you are using the default. ```bash /snap/bin/couchdb.remsh -n couchdb@localhost -c $COOKIE ``` + # Building this

Re: [PR] Snap updated to 3.3.2 [couchdb-pkg]

2023-11-07 Thread via GitHub
sklassen commented on code in PR #119: URL: https://github.com/apache/couchdb-pkg/pull/119#discussion_r1385878314 ## README-SNAP.md: ## @@ -251,30 +277,30 @@ you are using the default. ```bash /snap/bin/couchdb.remsh -n couchdb@localhost -c $COOKIE ``` + # Building this

Re: [PR] Snap updated to 3.3.2 [couchdb-pkg]

2023-11-07 Thread via GitHub
sklassen commented on code in PR #119: URL: https://github.com/apache/couchdb-pkg/pull/119#discussion_r1385881949 ## snap/snapcraft.yaml: ## @@ -1,106 +1,77 @@ +--- name: couchdb -version: 3.1.1 +version: 3.3.2 +type: app +base: core22 summary: Official Apache CouchDB snap -

Re: [PR] mango: add $beginsWith operator [couchdb]

2023-10-31 Thread via GitHub
willholley commented on PR #4810: URL: https://github.com/apache/couchdb/pull/4810#issuecomment-1786788940 @big-r81 oops on the premature merge - I'm too used to working with repos with branch protection enabled in GH and didn't think to manually check for the +1. -- This is an

Re: [PR] mango: add $beginsWith operator [couchdb]

2023-10-31 Thread via GitHub
willholley commented on PR #4810: URL: https://github.com/apache/couchdb/pull/4810#issuecomment-1786790838 curious that the failure didn't show in the PR checks - it definitely smells like a collation issue and likely an issue with the special characters used in the tests rather than the

Re: [PR] mango: add $beginsWith operator [couchdb]

2023-10-31 Thread via GitHub
big-r81 commented on PR #4810: URL: https://github.com/apache/couchdb/pull/4810#issuecomment-1786846495 @willholley np, only a reminder. We will add this review-branch-protection in the future. At the moment, we will get the "green" merge button, if the CI runs through... The

Re: [PR] mango: add $beginsWith operator [couchdb]

2023-10-31 Thread via GitHub
nickva commented on PR #4810: URL: https://github.com/apache/couchdb/pull/4810#issuecomment-1786501544 > It is not flaky, these errors happen every time -- but only on CentOS 7. Oh very interesting, definitely worth investigating. One thing that immediately jumps to mind is that

Re: [PR] mango: add $beginsWith operator [couchdb]

2023-10-30 Thread via GitHub
big-r81 commented on PR #4810: URL: https://github.com/apache/couchdb/pull/4810#issuecomment-1785861835 @pgj It was only a small friendly reminder  btw, this reminds me of #4772... -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[PR] Remove installation via snap package [couchdb]

2023-10-30 Thread via GitHub
big-r81 opened a new pull request, #4826: URL: https://github.com/apache/couchdb/pull/4826 Currently there is no maintainer of the snap package. Removing the docs until new maintainer is found. Closes https://github.com/apache/couchdb-pkg/issues/114. -- This is an automated

Re: [PR] mango: add $beginsWith operator [couchdb]

2023-10-30 Thread via GitHub
pgj commented on PR #4810: URL: https://github.com/apache/couchdb/pull/4810#issuecomment-1785880172 Locally (with Dreyfus/Clouseau enabled) `make mango-test` works for me. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

Re: [PR] mango: add $beginsWith operator [couchdb]

2023-10-30 Thread via GitHub
pgj commented on PR #4810: URL: https://github.com/apache/couchdb/pull/4810#issuecomment-1785849980 @big-r81 we have been in contact with @willholley about the PR out-of-band. On merging the PR he was aware that I did not have any more pending comments on the contents. I wanted to

Re: [PR] Remove installation via snap package [couchdb]

2023-10-30 Thread via GitHub
big-r81 commented on PR #4826: URL: https://github.com/apache/couchdb/pull/4826#issuecomment-1786067232 @sblaisot I'm happy to see your contributions to the snap package in the next days... -- This is an automated message from the Apache Git Service. To respond to the message, please log

Re: [I] compaction_log_level not working on Couchdb version 3.3.2 [couchdb]

2023-10-30 Thread via GitHub
nickva commented on issue #4815: URL: https://github.com/apache/couchdb/issues/4815#issuecomment-1785168773 Logically they sort of make sense in separate sections as they are logs coming from different areas. -- This is an automated message from the Apache Git Service. To respond to the

Re: [PR] mango: add $beginsWith operator [couchdb]

2023-10-30 Thread via GitHub
willholley commented on code in PR #4810: URL: https://github.com/apache/couchdb/pull/4810#discussion_r1375875992 ## src/mango/test/03-operator-test.py: ## @@ -141,6 +142,38 @@ def test_exists_false_returns_missing_but_not_null(self): for d in docs:

Re: [PR] mango: add $beginsWith operator [couchdb]

2023-10-30 Thread via GitHub
willholley commented on code in PR #4810: URL: https://github.com/apache/couchdb/pull/4810#discussion_r1375881371 ## src/docs/src/api/database/find.rst: ## @@ -754,8 +760,10 @@ can itself be another operator with arguments of its own. This enables us to build up more complex

Re: [I] compaction_log_level not working on Couchdb version 3.3.2 [couchdb]

2023-10-30 Thread via GitHub
big-r81 commented on issue #4815: URL: https://github.com/apache/couchdb/issues/4815#issuecomment-1784786455 @nickva Would it make sense to consolidate this into one section (e.g. [smoosh]) only? -- This is an automated message from the Apache Git Service. To respond to the message,

Re: [PR] mango: add $beginsWith operator [couchdb]

2023-10-31 Thread via GitHub
pgj commented on PR #4810: URL: https://github.com/apache/couchdb/pull/4810#issuecomment-1787188105 > it definitely smells like a collation issue and likely an issue with the special characters used in the tests In my understanding, the two failing test cases do not feature any

Re: [PR] mango: add $beginsWith operator [couchdb]

2023-10-31 Thread via GitHub
big-r81 commented on PR #4810: URL: https://github.com/apache/couchdb/pull/4810#issuecomment-1787391987 @pgj can you test on your local CentOS vm the state before the PR? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[I] Need to limit access to Fauxton UI to localhost, but maintain remote API access. [couchdb]

2023-10-31 Thread via GitHub
estoT1 opened a new issue, #4827: URL: https://github.com/apache/couchdb/issues/4827 Need to limit access to Fauxton UI to localhost, but maintain remote API access. Do I understand correctly that the Fauxton UI cannot be disabled or limited to localhost separately from the other

Re: [PR] mango: add $beginsWith operator [couchdb]

2023-10-31 Thread via GitHub
pgj commented on PR #4810: URL: https://github.com/apache/couchdb/pull/4810#issuecomment-1787413943 It is the derived string upper limit that does not work. Consider the following: ```erlang > Lower = <<"A">>. <<"A">>. > Upper = <>. <<"A\377">>. ``` On CentOS

Re: [PR] Wait for compacted indexes to flip. [couchdb]

2023-10-23 Thread via GitHub
nickva merged PR #4821: URL: https://github.com/apache/couchdb/pull/4821 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[PR] Upgrade MochiWeb to v3.2.1 [couchdb]

2023-10-23 Thread via GitHub
big-r81 opened a new pull request, #4822: URL: https://github.com/apache/couchdb/pull/4822 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe,

Re: [PR] Fix flaky windows test [couchdb]

2023-10-24 Thread via GitHub
big-r81 commented on PR #4406: URL: https://github.com/apache/couchdb/pull/4406#issuecomment-1777990429 Adding some other flaky tests: ``` module 'smoosh_persist' smoosh_persist:151: persist_unpersist_test_ (t_write_read_delete)...[0.015 s] ok smoosh_persist:152:

Re: [PR] Config delete reload [couchdb]

2023-10-26 Thread via GitHub
nickva commented on code in PR #4813: URL: https://github.com/apache/couchdb/pull/4813#discussion_r1372634840 ## src/config/src/config.erl: ## @@ -252,7 +251,7 @@ init(IniFiles) -> ets:new(?MODULE, [named_table, set, protected, {read_concurrency, true}]), lists:map(

Re: [PR] Config delete reload [couchdb]

2023-10-26 Thread via GitHub
nickva commented on code in PR #4813: URL: https://github.com/apache/couchdb/pull/4813#discussion_r1372634840 ## src/config/src/config.erl: ## @@ -252,7 +251,7 @@ init(IniFiles) -> ets:new(?MODULE, [named_table, set, protected, {read_concurrency, true}]), lists:map(

Re: [PR] Config delete reload [couchdb]

2023-10-26 Thread via GitHub
nickva commented on code in PR #4813: URL: https://github.com/apache/couchdb/pull/4813#discussion_r1372634840 ## src/config/src/config.erl: ## @@ -252,7 +251,7 @@ init(IniFiles) -> ets:new(?MODULE, [named_table, set, protected, {read_concurrency, true}]), lists:map(

Re: [PR] Config delete reload [couchdb]

2023-10-26 Thread via GitHub
pgj commented on code in PR #4813: URL: https://github.com/apache/couchdb/pull/4813#discussion_r1373082694 ## src/config/test/config_tests.erl: ## @@ -441,8 +437,8 @@ should_write_changes(_, _) -> ?assertEqual("5986", config:get("httpd", "port")),

Re: [PR] Decouple offline hash strength from online [couchdb]

2023-10-26 Thread via GitHub
rnewson commented on PR #4814: URL: https://github.com/apache/couchdb/pull/4814#issuecomment-1781301946 noting I just rebased after making an important internal change. the salt value for the internal cache is now the same as the one from the users doc. This ensures that the entry is not

Re: [PR] Decouple offline hash strength from online [couchdb]

2023-10-26 Thread via GitHub
rnewson commented on PR #4814: URL: https://github.com/apache/couchdb/pull/4814#issuecomment-1781298717 @nickva I've made the cache a configurable option. for deployments that are confident they avoid basic auth they can disable it without negative consequences. -- This is an automated

Re: [PR] Decouple offline hash strength from online [couchdb]

2023-10-26 Thread via GitHub
nickva commented on PR #4814: URL: https://github.com/apache/couchdb/pull/4814#issuecomment-1781372183 > I've made the cache a configurable option. for deployments that are confident they avoid basic auth they can disable it without negative consequences Makes sense. Also,

Re: [PR] mango: add $beginsWith operator [couchdb]

2023-10-26 Thread via GitHub
willholley commented on code in PR #4810: URL: https://github.com/apache/couchdb/pull/4810#discussion_r1373322353 ## src/mango/src/mango_selector_text.erl: ## @@ -142,6 +142,10 @@ convert(Path, {[{<<"$exists">>, ShouldExist}]}) -> true -> FieldExists; false ->

Re: [PR] mango: add $beginsWith operator [couchdb]

2023-10-26 Thread via GitHub
willholley commented on PR #4810: URL: https://github.com/apache/couchdb/pull/4810#issuecomment-1781399634 @pgj I think I've addressed all the comments, though will need to squash the commits -- This is an automated message from the Apache Git Service. To respond to the message, please

Re: [PR] Decouple offline hash strength from online [couchdb]

2023-10-26 Thread via GitHub
rnewson commented on PR #4814: URL: https://github.com/apache/couchdb/pull/4814#issuecomment-1781799017 hm, on FIPS-140 the fundamental algorithms (the SHA family) are invoked from openssl, which in such an environment is where the FIPS-140 "compliance" comes from. It will prevent the use

Re: [PR] Decouple offline hash strength from online [couchdb]

2023-10-26 Thread via GitHub
rnewson commented on PR #4814: URL: https://github.com/apache/couchdb/pull/4814#issuecomment-1781811900 (of course our current PBKDF2 implementation is bespoke anyway) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

Re: [PR] Enable branch protection for main branch [couchdb-ci]

2023-11-05 Thread via GitHub
nickva commented on PR #61: URL: https://github.com/apache/couchdb-ci/pull/61#issuecomment-1793841273 That’s fine we can try it out. I was just saying by the time the pr is made buildx would have uploaded the images already, so horse had left the barn already, as it were  -- This is an

Re: [PR] Enable branch protection for main branch [couchdb-ci]

2023-11-05 Thread via GitHub
big-r81 commented on PR #61: URL: https://github.com/apache/couchdb-ci/pull/61#issuecomment-1793844696 Maybe this sounds like we should automate this into Jenkins / GH Actions to run buildx only from there… -- This is an automated message from the Apache Git Service. To respond to the

[PR] feat: add Clouseau to the Docker images [couchdb-ci]

2023-11-05 Thread via GitHub
pgj opened a new pull request, #62: URL: https://github.com/apache/couchdb-ci/pull/62 This change makes it possible to run tests that require the Search functionality to be configured. Therefore the test coverage could be increased and issues could be caught with the related applications,

Re: [PR] Add Debian Bookworm [couchdb-ci]

2023-11-05 Thread via GitHub
nickva commented on PR #58: URL: https://github.com/apache/couchdb-ci/pull/58#issuecomment-1793995721 The image building is failing: ``` Reading database ... 27059 files and directories currently installed.) 133.2 Preparing to unpack .../nodejs_18.17.1-deb-1nodesource1_amd64.deb

Re: [PR] Add Debian Bookworm [couchdb-ci]

2023-11-05 Thread via GitHub
nickva commented on code in PR #58: URL: https://github.com/apache/couchdb-ci/pull/58#discussion_r1382643308 ## bin/apt-dependencies.sh: ## @@ -126,13 +125,13 @@ mkdir -p /usr/share/lintian/profiles/couchdb chmod 0755 /usr/share/lintian/profiles/couchdb if [[

Re: [PR] Add Debian Bookworm [couchdb-ci]

2023-11-06 Thread via GitHub
nickva commented on PR #58: URL: https://github.com/apache/couchdb-ci/pull/58#issuecomment-1795029033 Good find, @big-r81. That is tempting to install node directly from the repo. Well let's see if it's the one to blame, I think python is stopping the build as you found out.

Re: [PR] Add Debian Bookworm [couchdb-ci]

2023-11-06 Thread via GitHub
big-r81 commented on PR #58: URL: https://github.com/apache/couchdb-ci/pull/58#issuecomment-1794825015 Okay, it seems that there is a [change ](https://www.debian.org/releases/bookworm/amd64/release-notes/ch-information.en.html#python3-pep-668) in Debian Bookworm. We could also install

Re: [PR] Snap updated to 3.3.2 [couchdb-pkg]

2023-11-06 Thread via GitHub
big-r81 commented on code in PR #119: URL: https://github.com/apache/couchdb-pkg/pull/119#discussion_r1383879873 ## snap/snapcraft.yaml: ## @@ -1,106 +1,77 @@ +--- name: couchdb -version: 3.1.1 +version: 3.3.2 +type: app +base: core22 summary: Official Apache CouchDB snap - a

Re: [PR] feat: add Clouseau to the Docker images [couchdb-ci]

2023-11-06 Thread via GitHub
pgj commented on code in PR #62: URL: https://github.com/apache/couchdb-ci/pull/62#discussion_r1383883215 ## bin/install-clouseau.sh: ## @@ -0,0 +1,129 @@ +#!/usr/bin/env bash + +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license

Re: [PR] feat: add Clouseau to the Docker images [couchdb-ci]

2023-11-06 Thread via GitHub
pgj commented on code in PR #62: URL: https://github.com/apache/couchdb-ci/pull/62#discussion_r1383886408 ## bin/install-dependencies.sh: ## @@ -94,6 +98,9 @@ case "${OSTYPE}" in ERLANGVERSION=${ERLANGVERSION} ${SCRIPTPATH}/yum-erlang.sh

Re: [PR] feat: add Clouseau to the Docker images [couchdb-ci]

2023-11-06 Thread via GitHub
big-r81 commented on code in PR #62: URL: https://github.com/apache/couchdb-ci/pull/62#discussion_r1383890168 ## bin/install-clouseau.sh: ## @@ -0,0 +1,129 @@ +#!/usr/bin/env bash + +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license

Re: [PR] Decouple offline hash strength from online [couchdb]

2023-11-06 Thread via GitHub
nickva commented on code in PR #4814: URL: https://github.com/apache/couchdb/pull/4814#discussion_r1383718427 ## rebar.config.script: ## @@ -157,7 +157,8 @@ DepDescs = [ {jiffy,"jiffy",{tag, "CouchDB-1.0.9-2"}}, {mochiweb, "mochiweb",

Re: [PR] Decouple offline hash strength from online [couchdb]

2023-11-06 Thread via GitHub
rnewson commented on code in PR #4814: URL: https://github.com/apache/couchdb/pull/4814#discussion_r1383684384 ## rebar.config.script: ## @@ -157,7 +157,8 @@ DepDescs = [ {jiffy,"jiffy",{tag, "CouchDB-1.0.9-2"}}, {mochiweb, "mochiweb",

Re: [PR] Decouple offline hash strength from online [couchdb]

2023-11-06 Thread via GitHub
nickva commented on code in PR #4814: URL: https://github.com/apache/couchdb/pull/4814#discussion_r1383534528 ## rebar.config.script: ## @@ -157,7 +157,8 @@ DepDescs = [ {jiffy,"jiffy",{tag, "CouchDB-1.0.9-2"}}, {mochiweb, "mochiweb",

Re: [PR] Decouple offline hash strength from online [couchdb]

2023-11-06 Thread via GitHub
rnewson commented on code in PR #4814: URL: https://github.com/apache/couchdb/pull/4814#discussion_r1383786902 ## rebar.config.script: ## @@ -157,7 +157,8 @@ DepDescs = [ {jiffy,"jiffy",{tag, "CouchDB-1.0.9-2"}}, {mochiweb, "mochiweb",

Re: [PR] feat: add Clouseau to the Docker images [couchdb-ci]

2023-11-06 Thread via GitHub
big-r81 commented on code in PR #62: URL: https://github.com/apache/couchdb-ci/pull/62#discussion_r1383813350 ## bin/install-clouseau.sh: ## @@ -0,0 +1,129 @@ +#!/usr/bin/env bash + +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license

Re: [PR] Add Debian Bookworm [couchdb-ci]

2023-11-06 Thread via GitHub
big-r81 commented on PR #58: URL: https://github.com/apache/couchdb-ci/pull/58#issuecomment-1795947334 > For python perhaps we should just install with `--break-system-packages` since we're building a CI image not a general purpose dev? I think this would be okay. Can you try this before

Re: [PR] Snap updated to 3.3.2 [couchdb-pkg]

2023-11-06 Thread via GitHub
big-r81 commented on code in PR #119: URL: https://github.com/apache/couchdb-pkg/pull/119#discussion_r1383830141 ## README-SNAP.md: ## @@ -251,30 +277,30 @@ you are using the default. ```bash /snap/bin/couchdb.remsh -n couchdb@localhost -c $COOKIE ``` + # Building this snap

Re: [PR] feat: add Clouseau to the Docker images [couchdb-ci]

2023-11-06 Thread via GitHub
big-r81 commented on code in PR #62: URL: https://github.com/apache/couchdb-ci/pull/62#discussion_r1383816591 ## bin/install-dependencies.sh: ## @@ -94,6 +98,9 @@ case "${OSTYPE}" in ERLANGVERSION=${ERLANGVERSION} ${SCRIPTPATH}/yum-erlang.sh

Re: [PR] Add Debian Bookworm [couchdb-ci]

2023-11-06 Thread via GitHub
nickva commented on PR #58: URL: https://github.com/apache/couchdb-ci/pull/58#issuecomment-1795951398 I'll give it a try -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment.

Re: [PR] Snap updated to 3.3.2 [couchdb-pkg]

2023-11-06 Thread via GitHub
big-r81 commented on code in PR #119: URL: https://github.com/apache/couchdb-pkg/pull/119#discussion_r1383837641 ## README-SNAP.md: ## @@ -251,30 +277,30 @@ you are using the default. ```bash /snap/bin/couchdb.remsh -n couchdb@localhost -c $COOKIE ``` + # Building this snap

Re: [PR] feat: add Clouseau to the Docker images [couchdb-ci]

2023-11-06 Thread via GitHub
pgj commented on code in PR #62: URL: https://github.com/apache/couchdb-ci/pull/62#discussion_r1383952370 ## bin/install-clouseau.sh: ## @@ -0,0 +1,129 @@ +#!/usr/bin/env bash + +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license

Re: [PR] Add Debian Bookworm [couchdb-ci]

2023-11-06 Thread via GitHub
nickva commented on PR #58: URL: https://github.com/apache/couchdb-ci/pull/58#issuecomment-1796830961 It's still complaining ``` 69.4 Reading package lists... 169.9 Building dependency tree... 170.1 Reading state information... 170.3 The following additional packages will be

Re: [I] Spidermonkey 91 is EOL and unsupported [couchdb]

2023-10-30 Thread via GitHub
nickva commented on issue #4825: URL: https://github.com/apache/couchdb/issues/4825#issuecomment-1785389166 Thanks for the heads-up @RaitoBezarius. We're investigating using QuickJS https://github.com/apache/couchdb/issues/4448 but so far it hasn't made into a release yet. After the

[I] Spidermonkey 91 is EOL and unsupported [couchdb]

2023-10-30 Thread via GitHub
RaitoBezarius opened a new issue, #4825: URL: https://github.com/apache/couchdb/issues/4825 CouchDB 3.x series currently only support Spidermonkey 91, this dependency has been EOL for a long time now. In nixpkgs, we have a CouchDB packaging for the branch 3.x, we would like to package

Re: [PR] mango: add $beginsWith operator [couchdb]

2023-10-30 Thread via GitHub
pgj commented on code in PR #4810: URL: https://github.com/apache/couchdb/pull/4810#discussion_r1376324428 ## src/mango/test/03-operator-test.py: ## @@ -10,12 +10,13 @@ # License for the specific language governing permissions and limitations under # the License. +from

Re: [I] Spidermonkey 91 is EOL and unsupported [couchdb]

2023-10-30 Thread via GitHub
RaitoBezarius commented on issue #4825: URL: https://github.com/apache/couchdb/issues/4825#issuecomment-1785392792 @nickva I will investigate the security updates for 78, I am surprised but maybe that's the case! -- This is an automated message from the Apache Git Service. To respond to

[I] Couchdb Snap package stuck at couchdb 3.1.1 [couchdb-pkg]

2023-10-30 Thread via GitHub
sblaisot opened a new issue, #114: URL: https://github.com/apache/couchdb-pkg/issues/114 ## Expected Behavior Installing couchdb through snap packages following instructions at https://docs.couchdb.org/en/stable/install/snap.html install latest couchdb version (currently

Re: [I] Couch db throwing error "Error: normal. Reason: {gen_server,call,[<0.24064.2493>, {pread_iolist,327789}" [couchdb]

2023-10-25 Thread via GitHub
rummitta commented on issue #4819: URL: https://github.com/apache/couchdb/issues/4819#issuecomment-1779085227 hi, we are not getting the error now. The earlier couch db pod log seems to have rolled over. We will monitor this issue and get back in case it occurs again. -- This is an

Re: [I] Couchdb Snap package stuck at couchdb 3.1.1 [couchdb-pkg]

2023-10-30 Thread via GitHub
sblaisot commented on issue #114: URL: https://github.com/apache/couchdb-pkg/issues/114#issuecomment-1785725219 If you don't plan to publish updated snap package, at least the documentation should mention snap packages are deprecated. -- This is an automated message from the Apache Git

Re: [I] Couchdb Snap package stuck at couchdb 3.1.1 [couchdb-pkg]

2023-10-30 Thread via GitHub
rnewson commented on issue #114: URL: https://github.com/apache/couchdb-pkg/issues/114#issuecomment-1785754538 +1 to remove snap-related docs -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

Re: [PR] mango: add $beginsWith operator [couchdb]

2023-10-30 Thread via GitHub
willholley merged PR #4810: URL: https://github.com/apache/couchdb/pull/4810 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] mango: add $beginsWith operator [couchdb]

2023-10-30 Thread via GitHub
pgj commented on PR #4810: URL: https://github.com/apache/couchdb/pull/4810#issuecomment-1785802556 +1 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe,

Re: [PR] Mango query explain page [couchdb-fauxton]

2023-10-18 Thread via GitHub
Antonio-Maranhao commented on code in PR #1415: URL: https://github.com/apache/couchdb-fauxton/pull/1415#discussion_r1364693651 ## app/addons/documents/mango/components/ExplainPage.js: ## @@ -23,10 +26,175 @@ export default class ExplainPage extends Component {

Re: [PR] Config delete reload [couchdb]

2023-10-18 Thread via GitHub
jaydoane commented on code in PR #4813: URL: https://github.com/apache/couchdb/pull/4813#discussion_r1364906897 ## src/config/src/config.erl: ## @@ -326,6 +326,13 @@ handle_call({delete, Sec, Key, Persist, Reason}, _From, Config) -> "~p: [~s] ~s deleted for reason

[PR] Mango query explain page [couchdb-fauxton]

2023-10-17 Thread via GitHub
mharrigan26 opened a new pull request, #1415: URL: https://github.com/apache/couchdb-fauxton/pull/1415 ## Overview ## Testing recommendations ## GitHub issue number ## Related Pull Requests ## Checklist - [ ] Code is

Re: [PR] Config delete reload [couchdb]

2023-10-18 Thread via GitHub
nickva commented on code in PR #4813: URL: https://github.com/apache/couchdb/pull/4813#discussion_r1364857681 ## src/config/src/config.erl: ## @@ -326,6 +326,13 @@ handle_call({delete, Sec, Key, Persist, Reason}, _From, Config) -> "~p: [~s] ~s deleted for reason ~p",

[PR] Config delete reload [couchdb]

2023-10-18 Thread via GitHub
jaydoane opened a new pull request, #4813: URL: https://github.com/apache/couchdb/pull/4813 ## Overview Automatically reload disk default after config deletion. ## Testing recommendations ## Related Issues or Pull Requests ##

[GitHub] [couchdb] pgj commented on a diff in pull request #4766: Add `smoosh:status/1` to HTTP API

2023-09-19 Thread via GitHub
pgj commented on code in PR #4766: URL: https://github.com/apache/couchdb/pull/4766#discussion_r1330018636 ## src/docs/src/api/server/common.rst: ## @@ -1902,6 +1902,114 @@ See :ref:`Configuration of Prometheus Endpoint ` for details. Accept: text/plain Host:

[GitHub] [couchdb] rnewson commented on issue #4768: Encrypt the [ssl] password for the private key file.

2023-09-19 Thread via GitHub
rnewson commented on issue #4768: URL: https://github.com/apache/couchdb/issues/4768#issuecomment-1725447621 Obfuscation or masking provides no real protection (I note that the link you mentioned says the masking feature is "deprecated and exists only to maintain backward-compatibility"),

[GitHub] [couchdb-helm] clayvan commented on pull request #120: Add .Values.persistentVolume.selfManaged so PVCs can be managed

2023-09-19 Thread via GitHub
clayvan commented on PR #120: URL: https://github.com/apache/couchdb-helm/pull/120#issuecomment-1726758930 For what it's worth @yekibud I do think `existingClaims` supports most use cases, but I would agree its unintuitive to use as you have to dig through the templates to understand how

[GitHub] [couchdb] rnewson commented on pull request #4710: feat(`mango`): strict index selection

2023-09-22 Thread via GitHub
rnewson commented on PR #4710: URL: https://github.com/apache/couchdb/pull/4710#issuecomment-1731981572 Instruct > to [order](https://dictionary.cambridge.org/dictionary/english/order) or [tell](https://dictionary.cambridge.org/dictionary/english/tell) someone to do something,

[GitHub] [couchdb] big-r81 merged pull request #4766: Add new HTTP endpoint `/_node/_local/_smoosh/status`

2023-09-22 Thread via GitHub
big-r81 merged PR #4766: URL: https://github.com/apache/couchdb/pull/4766 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [couchdb] big-r81 closed issue #4705: add HTTP API for smoosh:status()

2023-09-22 Thread via GitHub
big-r81 closed issue #4705: add HTTP API for smoosh:status() URL: https://github.com/apache/couchdb/issues/4705 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

[GitHub] [couchdb] rnewson opened a new pull request, #4771: fix gradle deprecation warning

2023-09-21 Thread via GitHub
rnewson opened a new pull request, #4771: URL: https://github.com/apache/couchdb/pull/4771 ## Overview The automatic loading of test framework implementation dependencies has been deprecated. This is scheduled to be removed in Gradle 9.0. Declare the desired test framework directly

[GitHub] [couchdb] rnewson merged pull request #4771: fix gradle deprecation warning

2023-09-21 Thread via GitHub
rnewson merged PR #4771: URL: https://github.com/apache/couchdb/pull/4771 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [couchdb] big-r81 commented on issue #4768: Encrypt the [ssl] password for the private key file.

2023-09-22 Thread via GitHub
big-r81 commented on issue #4768: URL: https://github.com/apache/couchdb/issues/4768#issuecomment-1731236030 > Here is another example -> https://docs.confluent.io/platform/current/security/secrets-tutorial.html But in this example, what is the benefit? You would need to store the

[GitHub] [couchdb] big-r81 opened a new pull request, #4772: Extending branch protection rules of .asf.yaml

2023-09-22 Thread via GitHub
big-r81 opened a new pull request, #4772: URL: https://github.com/apache/couchdb/pull/4772 Add additional branch checks before merging a PR. 1. Require a linear history for better redability. 2. Disable "merge commits". 3. Require at least one approval. -- This is an

[GitHub] [couchdb] pgj opened a new pull request, #4774: fix(`docs`): amend description of `use_index` on `/{db}/_find`

2023-09-24 Thread via GitHub
pgj opened a new pull request, #4774: URL: https://github.com/apache/couchdb/pull/4774 The semantics of Mango's `use_index` query parameter has changed over time but this has not been reflected in the documentation, which causes a lot of confusion. The `use_index` parameter was

[GitHub] [couchdb] pgj commented on pull request #4710: feat(`mango`): strict index selection

2023-09-24 Thread via GitHub
pgj commented on PR #4710: URL: https://github.com/apache/couchdb/pull/4710#issuecomment-1732661208 Submitted #4774 to fix the documentation. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [couchdb] nickva commented on pull request #4769: Replace replicator open_doc_revs kaboom with something more descriptive

2023-09-18 Thread via GitHub
nickva commented on PR #4769: URL: https://github.com/apache/couchdb/pull/4769#issuecomment-1723820771 There is a `kaboom!` mentioned in the docs for the replicator protocol. But that one seems to be a generic example of a worker crashing:

[GitHub] [couchdb] estoT1 opened a new issue, #4768: Encrypt the [ssl] password for the private key file.

2023-09-18 Thread via GitHub
estoT1 opened a new issue, #4768: URL: https://github.com/apache/couchdb/issues/4768 Encrypt the [ssl] password for the private key file. ## Summary Currently the [ssl] private key file password is not getting encrypted. ## Desired Behaviour Encrypt the [ssl] password for

[GitHub] [couchdb] nickva commented on pull request #4766: Add `smoosh:status/1` to HTTP API

2023-09-18 Thread via GitHub
nickva commented on PR #4766: URL: https://github.com/apache/couchdb/pull/4766#issuecomment-1723898346 I wonder if it's worth leaving room in the API for additional status which is not channel related? That is, move the channel status under a "channels" sub-field?: ```

[GitHub] [couchdb] nickva opened a new pull request, #4769: Replace replicator open_doc_revs kaboom with something more descriptive

2023-09-18 Thread via GitHub
nickva opened a new pull request, #4769: URL: https://github.com/apache/couchdb/pull/4769 Use something more descriptive, especially since it can bubble up the user lvel when the replication job crashes. -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [couchdb] pavlov2000uk commented on issue #4768: Encrypt the [ssl] password for the private key file.

2023-09-20 Thread via GitHub
pavlov2000uk commented on issue #4768: URL: https://github.com/apache/couchdb/issues/4768#issuecomment-1727147629 Here is another example -> https://docs.confluent.io/platform/current/security/secrets-tutorial.html -- This is an automated message from the Apache Git Service. To respond

[GitHub] [couchdb] mikerhodes commented on pull request #4710: feat(`mango`): strict index selection

2023-09-20 Thread via GitHub
mikerhodes commented on PR #4710: URL: https://github.com/apache/couchdb/pull/4710#issuecomment-1727307260 > adding a note that for the new endpoints an index should not match a selector if it only has indexed fields where the selector does $regex or any other operator where neither

[GitHub] [couchdb] big-r81 commented on pull request #4766: Add `smoosh:status/1` to HTTP API

2023-09-18 Thread via GitHub
big-r81 commented on PR #4766: URL: https://github.com/apache/couchdb/pull/4766#issuecomment-1724155150 That sounds like a very good idea to be ready for adding additional status informations. -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [couchdb] nickva commented on a diff in pull request #4769: Replace replicator open_doc_revs kaboom with something more descriptive

2023-09-18 Thread via GitHub
nickva commented on code in PR #4769: URL: https://github.com/apache/couchdb/pull/4769#discussion_r1329050516 ## src/couch_replicator/src/couch_replicator_api_wrap.erl: ## @@ -284,7 +284,7 @@ open_doc_revs(#httpdb{retries = 0} = HttpDb, Id, Revs, Options, _Fun, _Acc) ->

[GitHub] [couchdb] nickva commented on a diff in pull request #4769: Replace replicator open_doc_revs kaboom with something more descriptive

2023-09-18 Thread via GitHub
nickva commented on code in PR #4769: URL: https://github.com/apache/couchdb/pull/4769#discussion_r1329178601 ## src/couch_replicator/src/couch_replicator_api_wrap.erl: ## @@ -284,7 +284,7 @@ open_doc_revs(#httpdb{retries = 0} = HttpDb, Id, Revs, Options, _Fun, _Acc) ->

[GitHub] [couchdb] rnewson commented on a diff in pull request #4769: Replace replicator open_doc_revs kaboom with something more descriptive

2023-09-18 Thread via GitHub
rnewson commented on code in PR #4769: URL: https://github.com/apache/couchdb/pull/4769#discussion_r1329032807 ## src/couch_replicator/src/couch_replicator_api_wrap.erl: ## @@ -284,7 +284,7 @@ open_doc_revs(#httpdb{retries = 0} = HttpDb, Id, Revs, Options, _Fun, _Acc) ->

[GitHub] [couchdb] rnewson commented on issue #4768: Encrypt the [ssl] password for the private key file.

2023-09-18 Thread via GitHub
rnewson commented on issue #4768: URL: https://github.com/apache/couchdb/issues/4768#issuecomment-1723997548 And how do you see the password being decrypted when the couchdb service starts? -- This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [couchdb] rnewson commented on pull request #4710: feat(`mango`): strict index selection

2023-09-20 Thread via GitHub
rnewson commented on PR #4710: URL: https://github.com/apache/couchdb/pull/4710#issuecomment-1727221556 adding a note that for the new endpoints an index should _not_ match a selector if it only has indexed fields where the selector does `$regex` or any other operator where neither

[GitHub] [couchdb] pgj commented on pull request #4774: fix(`docs`): amend description of `use_index` on `/{db}/_find`

2023-09-25 Thread via GitHub
pgj commented on PR #4774: URL: https://github.com/apache/couchdb/pull/4774#issuecomment-1733672686 @rnewson please note that I have included references to the commit history to support that the way `use_index` currently works is not a bug but a feature, a mitigation to the valid problem.

[GitHub] [couchdb] pgj commented on pull request #4710: feat(`mango`): strict index selection

2023-09-25 Thread via GitHub
pgj commented on PR #4710: URL: https://github.com/apache/couchdb/pull/4710#issuecomment-1734105026 Created #4775 for the problem with the `$regex` operator. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

<    1   2   3   4   5   6   7   8   9   10   >