cannot build go 1.9.2 on Solaris 11 x86

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
1.9.2
Does this issue reproduce with the latest release?
Yup.
What operating system and processor architecture are you using (go env)?
Solaris 11 amd64
What did you do?
export GOROOT_BOOTSTRAP=/usr/local/go (I have a working go.1.6.x there)
./all.bash
[...]
cmd/go/internal/test
cmd/go/internal/vet
# runtime/cgo
Assembler:
        "/var/tmp//cciDaGYk.s", line 17 : Syntax error
        Near line: ".section .note.GNU-stack,"",@progbits"
cmd/compile/internal/gc
cmd/compile/internal/amd64
cmd/compile/internal/mips
cmd/compile/internal/arm
cmd/compile/internal/arm64
cmd/compile/internal/mips64
cmd/compile/internal/s390x
cmd/compile/internal/ppc64
cmd/compile/internal/x86
cmd/compile
[root@hyperion src]#

When not using cgo:
Some test fail, thus everything fails:
ok      net/url 0.045s
ok      os      1.833s
--- FAIL: TestCredentialNoSetGroups (0.00s)
        exec_posix_test.go:20: error getting current user: user: Current not implemented on solaris/amd64
FAIL
FAIL    os/exec 2.541s
ok      os/signal       5.041s
--- FAIL: TestCurrent (0.00s)
        user_test.go:21: Current: user: Current not implemented on solaris/amd64 (got (*user.User)(nil))
--- FAIL: TestLookup (0.00s)
        user_test.go:68: Current: user: Current not implemented on solaris/amd64
--- FAIL: TestLookupId (0.00s)
        user_test.go:89: Current: user: Current not implemented on solaris/amd64
FAIL
FAIL    os/user 0.021s
[...]
ok      cmd/vet 3.902s
ok      cmd/vet/internal/cfg    0.019s
2017/12/19 15:19:20 Failed: exit status 1

What did you expect to see?
I'm expeting to see a valid go distribution after compilation's end.
What did you see instead?
I see nothing instead, compilation terminates.

Building Go with a bootrspping Go 1.4.x gives another set of errors:
[...everything is okay until here...]
ok      cmd/compile/internal/types      0.019s
ok      cmd/cover       4.061s
ok      cmd/doc 0.084s
ok      cmd/fix 0.040s
--- FAIL: TestDuplicateGlobalAsmSymbols (2.21s)
        go_test.go:310: running testgo [build -o /tmp/gotest077662300/p.exe p]
        go_test.go:323: standard error:
        go_test.go:324: # p
                /home/emz/src/go/pkg/tool/solaris_amd64/link: running gcc failed: exit status 1
                ld: fatal: relocation error: R_AMD64_64: file /tmp/go-link-314192241/000000.o: section: [10].rela.debug_info: offset: 0x19: relocation requires reference symbol
                ld: fatal: relocation error: R_AMD64_64: file /tmp/go-link-314192241/000000.o: section: [10].rela.debug_info: offset: 0x21: relocation requires reference symbol
                ld: fatal: relocation error: R_AMD64_64: file /tmp/go-link-314192241/000000.o: section: [10].rela.debug_info: offset: 0xb0: relocation requires reference symbol
                ld: fatal: relocation error: R_AMD64_64: file /tmp/go-link-314192241/000000.o: section: [10].rela.debug_info: offset: 0xb8: relocation requires reference symbol
                ld: fatal: symbol referencing errors
                collect2: ld returned 1 exit status

        go_test.go:333: go [build -o /tmp/gotest077662300/p.exe p] failed unexpectedly: exit status 2
FAIL
FAIL    cmd/go  64.386s
ok      cmd/go/internal/generate        0.024s
ok      cmd/go/internal/get     0.033s
ok      cmd/go/internal/load    0.051s
ok      cmd/go/internal/work    0.039s
ok      cmd/gofmt       0.128s
ok      cmd/internal/dwarf      0.041s
ok      cmd/internal/obj        0.021s
ok      cmd/internal/obj/arm64  0.035s
ok      cmd/internal/obj/x86    0.969s
ok      cmd/internal/objabi     0.031s
ok      cmd/internal/src        0.029s
ok      cmd/link        7.572s
ok      cmd/link/internal/ld    5.211s
--- FAIL: TestExternalLinkerCgoFile (1.87s)
        nm_test.go:109: building test executable failed: exit status 2 # command-line-arguments
                /home/emz/src/go/pkg/tool/solaris_amd64/link: running gcc failed: exit status 1
                ld: fatal: relocation error: R_AMD64_64: file /tmp/go-link-940018516/000000.o: section: [10].rela.debug_info: offset: 0x19: relocation requires reference symbol
                ld: fatal: relocation error: R_AMD64_64: file /tmp/go-link-940018516/000000.o: section: [10].rela.debug_info: offset: 0x21: relocation requires reference symbol
                ld: fatal: relocation error: R_AMD64_64: file /tmp/go-link-940018516/000000.o: section: [10].rela.debug_info: offset: 0xb0: relocation requires reference symbol
                ld: fatal: relocation error: R_AMD64_64: file /tmp/go-link-940018516/000000.o: section: [10].rela.debug_info: offset: 0xb8: relocation requires reference symbol
                ld: fatal: symbol referencing errors
                collect2: ld returned 1 exit status

FAIL
FAIL    cmd/nm  6.296s
ok      cmd/objdump     4.591s
ok      cmd/pack        3.939s
ok      cmd/trace       0.034s
ok      cmd/vendor/github.com/google/pprof/internal/binutils    0.024s
ok      cmd/vendor/github.com/google/pprof/internal/driver      0.289s
ok      cmd/vendor/github.com/google/pprof/internal/elfexec     0.019s
ok      cmd/vendor/github.com/google/pprof/internal/graph       0.023s
ok      cmd/vendor/github.com/google/pprof/internal/report      0.028s
ok      cmd/vendor/github.com/google/pprof/internal/symbolizer  0.028s
ok      cmd/vendor/github.com/google/pprof/internal/symbolz     0.023s
ok      cmd/vendor/github.com/google/pprof/profile      0.113s
ok      cmd/vendor/github.com/ianlancetaylor/demangle   0.059s
ok      cmd/vendor/golang.org/x/arch/arm/armasm 0.030s
ok      cmd/vendor/golang.org/x/arch/ppc64/ppc64asm     0.022s
ok      cmd/vendor/golang.org/x/arch/x86/x86asm 0.174s
ok      cmd/vet 3.762s
ok      cmd/vet/internal/cfg    0.019s
2017/12/19 16:39:13 Failed: exit status 1

Relocation errors in 1.5.x similar to those that I encountered in 1.9.2 were fixed there: https://github.com/golang/go/issues/14211
Looks like they are back.

/cc @binarycrusader

@drook, have you tried git master, which will soon become Go 1.10?

 I'll close this, then.


Optional affix search for click-in-text API

I also added a todo (which I will do), and a fixme (which gives me autistic screeching)...
Also some large diffs are from formatting of my IDE, I tried to use black instead. Looks like something is messed up:
C:\Users\Matt\PycharmProjects\cree-intelligent-dictionary>pipenv run python -m black CreeDictionary
Traceback (most recent call last):
  File "C:\Users\Matt\AppData\Local\Programs\Python\Python36-32\Lib\runpy.py", line 183, in _run_module_as_main
    mod_name, mod_spec, code = _get_module_details(mod_name, _Error)
  File "C:\Users\Matt\AppData\Local\Programs\Python\Python36-32\Lib\runpy.py", line 142, in _get_module_details
    return _get_module_details(pkg_main_name, error)
  File "C:\Users\Matt\AppData\Local\Programs\Python\Python36-32\Lib\runpy.py", line 109, in _get_module_details
    __import__(pkg_name)
  File "C:\Users\Matt\.virtualenvs\cree-intelligent-dictionary-aKpDmaTl\lib\site-packages\black\__init__.py", line 49, in <module>
    from dataclasses import dataclass, field, replace
ModuleNotFoundError: No module named 'dataclasses'

I'll figure it out later...

Codecov Report

Merging #571 (6ef935c) into master (531f0c0) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #571      +/-   ##
==========================================
+ Coverage   91.64%   91.66%   +0.02%
==========================================
  Files          73       73
  Lines        2644     2651       +7
==========================================
+ Hits         2423     2430       +7
  Misses        221      221

Impacted Files
Coverage Δ

CreeDictionary/API/models.py
82.78% <100.00%> (ø)

CreeDictionary/API/search.py
95.72% <100.00%> (+0.01%)
:arrow_up:

CreeDictionary/API/views.py
94.11% <100.00%> (ø)

CreeDictionary/tests/API_tests/test_views.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 531f0c0...6ef935c. Read the comment docs.


[FLINK-11466][e2e] Design the e2e java framework so that the Kafka streaming tests and Kafka SQL tests can running on it

What is the purpose of the change
Rewrite the e2e kafka related test by Java, so that we can move the FLINK-11463 forward.   it need to meet the following requirement:

Should abstract the Flink resources, and provide standalone implementation & distribution implementation, so that the test can run on different flink env.
Should abstract the Kafka resource,  and provide standalone implementation at least, can leave the distribution impl as dummy because seems we're not depending on it now.
Will provide the file cache later , not in this PR because it's seems big enough now, another PR will be better for reviewing.
Set different catalog for each e2e test case, so that we can configure to decide whether to run the given test in diff nightly case ( no need to run most of the e2e test in precommit env). it will be another PR.

Brief change log

4b43f87c14 Introduce the rough java framework for flink end-to-end testing
b523eace79 Define the KafkaDistribution and its behavior, so that the upper layer e2e java framework can depends on it.
876f0f86b2 Add the kafka streaming end-to-end tests
4108e40950 Abstract the KafkaResource so that the e2e test can run on both standalone cluster and distributed cluster, currently the distributed kafka resource is still in TODO state.

Verifying this change
This change added tests and can be verified as follows:
mvn package test \
-DdistDir=/Users/openinx/software/flink/build-target \
-De2e.flink.mode=localStandalone \
-De2e.kafka.mode=localStandalone \
-Dtest=TestSQLClientKafka,TestSQLClientKafka010,TestSQLClientKafka011,TestStreamingKafka,TestStreamingKafka010,TestStreamingKafka011  \
-pl org.apache.flink:flink-end-to-end-tests-common

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not documented)

Ping @tillrohrmann @zentol for reviewing ..

The e2e framework will build based on the previous PR from @zentol (https://github.com/apache/flink/pull/7605),  so close this one now. and I will fix all the conflicts in @zentol 's PR and create a new one. Thanks.


jb install fails with 0.7

What happened?
the command jb install<EMAIL_ADDRESS>generate a panic on rename
How to reproduce it (as minimally and precisely as possible):
jb init
jb install<EMAIL_ADDRESS>
error :
GET https://github.com/coreos/kube-prometheus/archive/4dbc23c923a8c1978e7daea6eb301623d75a9be1.tar.gz 200
GET https://github.com/kubernetes/kube-state-metrics/archive/e72315512a38653b19dcfe4429f93eadedc0ea96.tar.gz 200
GET https://github.com/prometheus/prometheus/archive/26d89b4b0776fe4cd5a3656dfa520f119a375273.tar.gz 200
GET https://github.com/prometheus/alertmanager/archive/193ebba04d1e70d971047e983a0b489112610460.tar.gz 200
GET https://github.com/etcd-io/etcd/archive/e8ba375032e8e48d009759dfb285f7812e7bcb8c.tar.gz 200
panic: rename /Users/hleclerc/tmp/jjj/vendor/.tmp/jsonnetpkg-github.com-etcd-io-etcd-contrib-mixin-e8ba375032e8e48d009759dfb285f7812e7bcb8c647930610/contrib/mixin /Users/hleclerc/tmp/jjj/vendor/github.com/etcd-io/etcd/contrib/mixin: no such file or directory

goroutine 1 [running]:
github.com/jsonnet-bundler/jsonnet-bundler/pkg.(*GitPackage).Install(0xc0000b20f0, 0x14ca7a0, 0xc0000b6030, 0xc0001e4e70, 0x25, 0xc0000b85e0, 0x1e, 0xc00044de30, 0x28, 0x0, ...)
	/Users/hleclerc/go/src/github.com/jsonnet-bundler/jsonnet-bundler/pkg/git.go:220 +0x1d56
github.com/jsonnet-bundler/jsonnet-bundler/pkg.download(0xc00051c410, 0x0, 0xc00044de30, 0x28, 0x0, 0x0, 0x0, 0x0, 0x0, 0xc0000b85e0, ...)
	/Users/hleclerc/go/src/github.com/jsonnet-bundler/jsonnet-bundler/pkg/packages.go:289 +0x1aa
github.com/jsonnet-bundler/jsonnet-bundler/pkg.ensure(0xc0004efe90, 0xc0000b85e0, 0x1e, 0xc00009bf80, 0x0, 0x0, 0x0)
	/Users/hleclerc/go/src/github.com/jsonnet-bundler/jsonnet-bundler/pkg/packages.go:233 +0x6c8
github.com/jsonnet-bundler/jsonnet-bundler/pkg.ensure(0xc00009bf20, 0xc0000b85e0, 0x1e, 0xc00009bf80, 0x1e, 0x0, 0x1)
	/Users/hleclerc/go/src/github.com/jsonnet-bundler/jsonnet-bundler/pkg/packages.go:259 +0xef6
github.com/jsonnet-bundler/jsonnet-bundler/pkg.Ensure(0xc00009bf20, 0x1, 0xc0000b85e0, 0x1e, 0xc00009bf80, 0x0, 0x0, 0x0)
	/Users/hleclerc/go/src/github.com/jsonnet-bundler/jsonnet-bundler/pkg/packages.go:55 +0x67
main.installCommand(0xc00001a044, 0x17, 0x14352ca, 0x6, 0xc000098fc0, 0x1, 0x1, 0x0, 0x0)
	/Users/hleclerc/go/src/github.com/jsonnet-bundler/jsonnet-bundler/cmd/jb/install.go:75 +0x74c
main.Main(0xc00008e058)
	/Users/hleclerc/go/src/github.com/jsonnet-bundler/jsonnet-bundler/cmd/jb/main.go:82 +0xb07
main.main()
	/Users/hleclerc/go/src/github.com/jsonnet-bundler/jsonnet-bundler/cmd/jb/main.go:37 +0x22

I experience the same since #980 has been merged. Commit c1130442d68bf4bfbd1544711b67c98a28f2a327 works for me.

I can reproduce the same issue.

I think that the etcd-mixin is not at the right place in the archive. it leaves in Documentation directory in tar.gz file. A mistake by the maintainer of etcd mixin maybe.

Etcd changed repository layout and now mixin is located in contrib/mixin. However, the issue is we are using that layout, but pinning to commit with an older one. https://github.com/prometheus-operator/kube-prometheus/pull/1009 should fix that.

Closing as #1009 was merged. Please reopen if this is still causing problems.


Using stars truncates the string.

When trying to mask 2345 23** **** 2345 I would assume this pattern would work: 0000 00** **** 0000. But it only works up to the last star: 2345 23** ****

bump


Add support for second players inputs

So far only lshift/tab is displayed in the input display, and nothing in the piano roll.

The piano roll can display any input of the second player.
The choice of inputs to show in the piano roll or the input display is still hardcoded.

New input display available, new bindings available.
Up-to-date documentation.
Testing could be a bit more advanced, but everything seems to work fine.
Nothing is changed if none of the second player inputs are used.


Volunteer

Updated Board members component
Updated Volunteer list
Made small design changes to accommodate volunteers and board members
made the component responsive

Great work @AshvinjeetSingh 🔥
The styling and the component is consistent with the site design.
I don't have any review comments. So merging the PR to dev.


Better work with cassandra image

add script for liveness probe
* Remove Env var injected in Cassandra Pod since they are onbly used in Bootstrap Init Container
* Disable entrypoint for native docker image since it breaks configuration done by Bootstrap Container

rebasing on master


Compile error due to new assertThatThrownBy

Describe the bug
Compile error in Eclipse 2023-12: The method assertThatThrownBy(ThrowableAssert.ThrowingCallable) is ambiguous for the type AmbigiousCallTest.
The only change is the upgrade from 3.24.2 to 3.25. In 3.24.2 the same code was fine.

assertj core version: 3.25

Test case reproducing the bug
Don't run the below code, but just compile it. At least ECJ doesn't like it.
public class AmbigiousCallTest {

    @Test
    void ambigiousCall() throws Exception {
        assertThatThrownBy(AmbigiousCallTest::verify)
                .hasMessageContaining("No objects are given");
    }

    public static void verify(final Object... objects) {
        // foo
    }

}

Probably related to #3043. Workaround: Extract the method reference to a local variable. The type information of that local variable is sufficient to choose one of the two methods.

Thanks for reporting it, @Bananeweizen.
We're in discussion with the team if we should revert cf06398587da23ac530b03a941e0efe6cf29aef7 as it introduced a breaking change we didn't spot before.

My code also doesn't compile after upgrading to AssertJ 3.25.0,
BUT
@scordio I would like to mention that this is not a big deal for me.
I just need to replace method reference (which is not really convenient to read):
assertThatThrownBy(Selenide::confirm)
      .isInstanceOf(AlertNotFoundError.class)

to a "standard" lambda (which is better readable to me):
assertThatThrownBy(() -> Selenide.confirm())
      .isInstanceOf(AlertNotFoundError.class)

We got hit by this in Spring Framework as well.
https://github.com/spring-projects/spring-framework/commit/ffddbb586e60ce1b66a3b962612636db9c61ed81#diff-58b8a6574dcffc60614d0727c28b36039f5bc28a04143486722d908a24ff55c7

FWIW, if you do intend to keep ThrowingCallableWithValue, note that most people typically call that a ThrowingSupplier. 😉

Team decision: cf06398587da23ac530b03a941e0efe6cf29aef7 should be reverted and #3043 should be evaluated again together with #1652 / #2519.

While the workaround isn't so bad with Java, it seemed a little more painful with Groovy, so appreciate the revert for now.
(disclaimer: not a Groovy expert so correction is fine if there is a more elegant way to do this ... short of falling back to Java-style syntax!)
assertThatThrownBy { doThing() }

became
assertThatThrownBy({ doThing() } as ThrowableAssert.ThrowingCallableWithValue)

Is it worth adding test cases to catch this in the future?

@ascopes already done, see #3318.

great, thanks for the quick fixes!

@scordio This is a bit out of scope, but could you imagine to build and test some other popular projects with every commit to master, using the freshly compiled assertj version? Similar to how the OpenJDK quality outreach program builds and tests 30(?) popular Java libraries regularly with their HEAD revision of Java, to verify they still work fine. That way regressions could be found soon after they are introduced, independent of a release.

That's an interesting idea, @Bananeweizen!
We're involved in the quality outreach program but I wasn't aware that OpenJDK is pulling & building projects. Until now, we provided manual feedback on which versions we test against.
Would you have any references about it?
Anyway, it's worth investigating how to do it for some projects like Spring and JUnit. We'll discuss it with the team.
Thanks again for bringing this up!

Agree this sounds like a useful idea.
As an example, the library I referenced above flagged this issue, so could be a candidate for this sort of thing potentially.


Resource availability issues

I am running the PyTorch_nyctaxi example on a node with 80 CPUs, each with 20 physical cores. If I set the number of executors to a number larger than 16 and the number of workers in TorchEstimator to more than 20, the code won't run and I would get resource unavailable errors, such as these ones:
(raylet) terminate called after throwing an instance of 'std::system_error'
(raylet)   what():  Resource temporarily unavailable
2021-02-26 11:42:09,982	WARNING worker.py:1090 -- The node with node id e7d998b5c895816e6446685482cb0392ad76245a8dab474562b232d9 has been marked dead because the detector has missed too many heartbeats from it. This can happen when a raylet crashes unexpectedly or has lagging heartbeats.
Obviously, I am utilizing a small portion of the computing resources I have available on the node. I do not specify the num_cpus to use in ray.init(), so it should automatically detect all 80.
Any idea why is this happening?

Hi @yanivg10, you mean you have a cluster with 20 workers and each worker has 80 cores? The raydp will occupy num_executors * executor_cores cpus on the cluster.
So you want to release the spark cluster resources when training model, right?

My cluster is composed of one node with 80 CPUs, each has 20 cores. But RayDP does not allow me to use more than the following setting (I am getting resource availability errors when increasing num_executors):
num_executors = 16
cores_per_executor = 1
memory_per_executor = "1GB"
spark = raydp.init_spark(app_name, num_executors, cores_per_executor, memory_per_executor)
As for your question, how do I free the spark cluster resources in order to use these CPUs for training?

@yanivg10 , did you check the Ray dashboard? How many cores did Ray detect after running ray.init()? RayDP will create actors based on the parameters you gave to raydp.init_spark. It sounds like Ray doesn't have enough resource to meet the requirement.

Thanks. I was able to find a configuration that uses my resources correctly without throwing errors.
Now I want to run RayDP on a cluster of multiple nodes. Do you have instructions on configuring RayDP on multiple nodes?

You can refer to https://docs.ray.io/en/master/cluster/quickstart.html

close as stale


Add implementation and contract tests for new AWS resources

Description of changes:
Support instrumentation for the following attributes. See this Python PR for the source of feature parity.

For SNS, extract TopicArn from request body and add to span as aws.sns.topic.arn.
For Secrets Manager, extract either SecretId from request body or ARN from response body and add to span as aws.secretsmanager.secret.arn.
For Step Functions, extract activityArn or  stateMachineArn from request body, and add to span as aws.stepfunctions.activity.arn or aws.stepfunctions.state_machine.arn respectively.
For Lambda, extract UUID from request body for event source mapping functions, and add to span as aws.lambda.resource_mapping.id.

In addition, the new resource attribute aws.remote.resource.cfn.primary.identifier is also added for each new resource.
E2E tests show that the new attributes and their associated CFN primary identifiers have been correctly populated for each service.
SNS:

Secrets Manager:

Step Functions:

Lambda:

Contract tests verify the expected span attribute values with example function calls that involve each resource. https://github.com/aws-observability/aws-otel-dotnet-instrumentation/actions/runs/12127063253/job/33813330563?pr=143
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Both this PR and PR #142 add aws.remote.resource.cfn.primary.identifier. Is there any dependency between them? Should PR #142 be merged firstly?

Both this PR and PR #142 add aws.remote.resource.cfn.primary.identifier. Is there any dependency between them? Should PR #142 be merged firstly?

Yes, #142 should be merged first as it originally implements aws.remote.resource.cfn.primary.identifier, and this PR was rebased on #142 so that this CFN Identifier attribute could be applied for the new resources.


Alexa Media Player - Action Required

just upgraded
I click CONFIGURE
do the Captcha thing
then I get the TXT on phone
click the Link approve it

when hit SUBMIT on above screen nothing happen
what do i miss
If i click the x it goes away but the if i hit CONFIGURE again I get above screen up.
IF do a restart of HA we back to the beginning again

Fill out the template. You are required to have 2FA already enabled to use the component. I will not debug issues if your account does not have 2FA.

Please see the instructions.

THANK YOU
SEE YOU MUST READ MANUAL


Support Rich Text Label [Feature Request]

More of a feature request, but it'd be nice if this had Rich Text Label support.
Tried forking it myself, but a lot of logic fails with rich text label because of how number of lines are determined in auto wrap between label and rich text label.

I will love this feature, I also tried to code it myself, but the problem I had is that the markup is displayed before the text and then when the text is displayed, the markup hides itself
I am trying to find a way to hide the markup completely but honestly I have not found a good way, but I think it is due to the logic of the code
since, at least what I understand, it adds the letters one by one at the same time that it shows them, so the node of the RichTextLabel does not have time to say "Oh wait, this text must be blue" for example
However, I am a bit new to programming, so there is still hope (I think) hopefully this feature will be added soon:,)

Hello, I have no idea how this works because I haven't looked, but you said that you were new to programming, and I really like this project, so let me tell you how I would do it based on what's been said:
You said that it currently gets the characters 1 by 1 (from a string, I'm assuming). I believe that you should probably parse the string into some abstract syntax tree (AST), which is a fancy way of saying "data structure" or "object", and in this case the outer object would be a list (or array) of these data structures: each segment of the string with identical formatting would have an object with a string as you currently have , but it would also have all formating information, then you cam just loop over those objects and render them 1 by 1.


fix: small typo

Just noticed a small open quote in the README while looking through the repo. Thank you for all the work!

This seems to be a duplicated of #476

This seems to be a duplicated of #476

Nice

Thank you! Sorry for being slow on this, I merged another duplicate pull request.


Adorama CAPTCHA Issues

I know there was a previously closed case touching on this, but I'm now getting nothing but CAPTCHA failures on the Adorama container running:
`
I2020-12-23 17:50:23,272 [drm_1] scraper initialized for https://www.adorama.com/astr3080o10g.html
I2020-12-23 17:50:23,272 [drm_2] scraper initialized for https://www.adorama.com/astrx308010g.html
I2020-12-23 17:50:23,273 [drm_3] scraper initialized for https://www.adorama.com/ev10g53881kr.html
I2020-12-23 17:50:23,273 [drm_4] scraper initialized for https://www.adorama.com/ev10g53885kr.html
I2020-12-23 17:50:23,273 [drm_5] scraper initialized for https://www.adorama.com/msig380gxt1.html
I2020-12-23 17:50:23,273 [drm_6] scraper initialized for https://www.adorama.com/png30801tfxb.html
I2020-12-23 17:50:23,273 [drm_7] scraper initialized for https://www.adorama.com/pnv301tfxmpb.html
W2020-12-23 17:51:26,105 [root] unable to save screenshot of webpage: Message: timeout: Timed out receiving message from renderer: 10.000
(Session info: chrome=83.0.4103.116)
W2020-12-23 17:51:26,160 [drm_1] access denied, got a CAPTCHA
W2020-12-23 17:52:28,052 [root] unable to save screenshot of webpage: Message: timeout: Timed out receiving message from renderer: 10.000
(Session info: chrome=83.0.4103.116)
W2020-12-23 17:52:28,100 [drm_2] access denied, got a CAPTCHA
W2020-12-23 17:53:29,986 [root] unable to save screenshot of webpage: Message: timeout: Timed out receiving message from renderer: 10.000
(Session info: chrome=83.0.4103.116)
W2020-12-23 17:53:30,030 [drm_3] access denied, got a CAPTCHA
W2020-12-23 17:54:31,878 [root] unable to save screenshot of webpage: Message: timeout: Timed out receiving message from renderer: 10.000
(Session info: chrome=83.0.4103.116)
W2020-12-23 17:54:31,940 [drm_4] access denied, got a CAPTCHA`
Any ideas?

Same issue


add Neura blockchain

Can you add "codebase" and "versions" (in "versions" array you'll just start with the same current information in "codebase") for all three networks (main, testnet, devnet)?

hey @reversesigh. The chain development is in progress and the repo is still private. I've added the version only. Other information will be added later

@reversesigh

Is only the source private or are binaries private as well?
Yes, binaries are private. We would not like to publish them until the security audit is completed.


Access Denied when deploying assessment container

The actual ip address of mysql container is <IP_ADDRESS> but for some reason it attemps to log into <IP_ADDRESS> during the deployment. Enrollement containers have been deployed fine. However, when I deploy the enrollement db container, I get a message that the database already exists. I am able to shell to both mysql containers and log into the db with credentials identified in .env files.
Access denied for user<EMAIL_ADDRESS>(using password: YES)
Couldn't create 'disease_trakker_production' database. Please check your configuration.
rake aborted!
Mysql2::Error::ConnectionError: Access denied for user<EMAIL_ADDRESS>(using password: YES)
my .env file:
ISEASE_TRAKKER_DATABASE_PASSWORD=foo
DISEASE_TRAKKER_DATABASE_USERNAME=disease_trakker
DISEASE_TRAKKER_DATABASE_HOST=mysql-assessment
DISEASE_TRAKKER_DATABASE_NAME=disease_trakker_production
MYSQL_USER=disease_trakker
MYSQL_PASSWORD=
MYSQL_ROOT_PASSWORD=
MYSQL_DATABASE=disease_trakker_production_assessment
SECRET_KEY_BASE=<my key base that comes from rake secret'>
SARA_ALERT_REPORT_MODE=true
SHOW_DEMO_WARNING=true

I'm a little confused - the title says that you have trouble deploying the assessment container but it seems like the issue is with the enrollment database?

Are you saying that the issue only appears when deploying enrollment & assessment containers at the same time?
Are you using the provided Production docker-compose file? I'll admit that the other Docker files are less exercised when deploying both assessment and enrollment as that's really a production thing.
Can you share some steps or commands you took to build the containers

I would ensure that all volumes are removed and containers are stopped after you encounter an error. Database already existing sounds like the volume has persisted and perhaps there is a lingering database container on the IP address in question.


 This was found with Cppcheck. I dunno whether the second comparison should just be removed or something else should be checked instead.

Fixed by commit 2647e2617e3.


New command: m365 pp pipeline stage get

Usage
m365 pp pipeline stage get
Description
Gets information about the specified Power Platform pipeline stage
Options

Option
Description

-i, --id<id>
The id of the stage to retrieve.

--pipelineId <pipelineId>
The id of the pipeline to retrieve the stage from.

-e, --environmentName <environmentName>
The name of the environment where the pipelines are located.

--asAdmin
Run the command as admin and retrieve Power Pages websites for environments you do not have explicitly assigned permissions to.

Examples
List given stages for the pipeline in the given Power Platform environment.
m365 pp pipeline stage get --id d87a7535-dd31-4437-bfe1-95340acd55c5 --pipelineId d87a7535-dd31-4437-bfe1-95340acd55c5 --environmentName Default-d87a7535-dd31-4437-bfe1-95340acd55c5

List give tstage for the pipeline in the given Power Platform environment as admin.
m365 pp pipeline stage get --id d87a7535-dd31-4437-bfe1-95340acd55c5 --pipelineId d87a7535-dd31-4437-bfe1-95340acd55c5 --environmentName Default-d87a7535-dd31-4437-bfe1-95340acd55c5 --asAdmin

Default properties
N/a
Additional Info
https://learn.microsoft.com/en-us/power-platform/developer/pipelines/table-reference#deploymentstage-deployment-stage

@appieschot is it possible to get pipeline stage by name ?
Other than that I have no further comments and we could open it up


PL locale - wrong day name returned as the first day of week

Hi,
I just noticed that when using 'pl' as locale, moment returns 'niedziela' (Sunday) as the first day of the week when correctly this should be 'poniedziałek' (Monday).
You can also find proof on wikipedia: https://en.wikipedia.org/wiki/Names_of_the_days_of_the_week#Days_numbered_from_Monday
To Reproduce and Expected behavior
test("beginning of week", () => {
  expect(
    moment()
      .locale("pl")
      .day(0)
      .format("dddd")
  ).toBe("poniedziałek");
});

Desktop (please complete the following information): and Moment-specific environment

OS: iOS; Browser: chrome;
my timezone: GMT +1
I'm using typescript

You asked for some console logging. I added the output results as comments below:
console.log((new Date()).toString()) \\Tue Mar 24 2020 12:28:35 GMT+0100 (czas środkowoeuropejski standardowy)
console.log((new Date()).toLocaleString()) \\24.03.2020, 12:28:35
console.log((new Date()).getTimezoneOffset()) \\ -60
console.log(navigator.userAgent) \\Mozilla/5.0 (darwin) AppleWebKit/537.36 (KHTML, like Gecko) jsdom/11.12.0
console.log(moment.version) \\2.24.0

You can always reconfigure your moment for rearraning days of week:
moment.locale('en', {
    week: {
        dow: 6
    }
})

thoughts @evoL ?

Hello!
In Moment, numbers representing days of week are always fixed no matter the locale — Sunday is 0, Monday is 1 etc. The docs for moment().day(Number) describe this.
This means that your example always asks for the translated name of Sunday and therefore works as expected. To return the locale-specific day of week, use moment().startOf('week'):
moment().locale('pl').startOf('week').format('dddd')

The above returns "poniedziałek" (Monday), as expected.


Match answer component LESQ-394

Description
Music year: 1954
-Refactors match answer component to fix spacing
Issue(s)
Fixes #LESQ-394
How to test

Go to https://deploy-preview-2220--oak-web-application.netlify.thenational.academy/teachers/programmes/english-primary-ks2/units/key-terminology-including-determiners-fronted-adverbials-and-parenthesis/lessons/determiners-articles-possessive-pronouns-quantifiers-and-demonstratives#exit-quiz

Screenshots
How it used to look (delete if n/a):
{screenshots}
How it should now look:
{screenshots}
Checklist

[ ] Added or updated tests where appropriate
[ ] Manually tested across browsers / devices
[ ] Considered impact on accessibility
[ ] Design sign-off
[ ] Approved by product owner
[ ] Does this PR update a package with a breaking change

:tada: This PR is included in version 1.336.3 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Update lodepng to 20201017 release

I replaced lodepng.h/.c files with the newest release (20201017).
I applied the same changes that were done to old lodepng (very few!) for lv_lib_png use.
The new lodepng release saves about 40KB of memory allocation (tested on 128x128pixel file).
Regards,

I've also tested it and worked well. Thank you very much for the update!


Add JakobDev.jdTextEdit

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.2 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

/azp run

/azp run

I believe the issue is heavy bandwidth throttling, possibly based on user agent. My sandbox sees a very slow download, and got stuck with an incomplete download at 28 MB of 43 MB. Meanwhile a browser on the same PC gets the package in a few seconds.

/azp run

@wingetbot waivers Add Validation-Unapproved-URL

/azp run


Feature/pubmed update

https://github.com/opentrials/opentrials/issues/200

@vitorbaptista please take a look

Looks good :+1:


Request: USD Icon

Great project!  Not sure on your views on this but it would be great if there were some fiat icons to go along with the alt pairs, specifically USD for me but I'm sure others wouldn't mind EUR, JPY etc..  Was checking out the noun project and was going to make a simple one but I'm sure with the source files you could make a much better fit.
Thanks again

What a great idea, thanks!
I have added USD, GBP, EUR, JPY, CNY, and RUB to start off with.
https://github.com/cjdowner/cryptocurrency-icons/commit/a7c59e1f54db772316c7f767ed8fed70ee481865

That's awesome, thanks!


Update reclaim-ai extension with support for new prioritization features, bug fixes, and authorship.

Description

Updates to multiple parts of the extension to support the new prioritization work that's available in beta (https://help.reclaim.ai/en/articles/8291694-priorities-overview)
Backed out recent changes around duplicate events which caused numerous customer support cases
Update to authorship as original author is no longer a Reclaim employee

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

@shkreios This PR reverts your recent changes that came in. We got some customer tickets reporting some issues, so we want to temporarily revert it. But I think your changes are good ones we should merge back in soon. We just need some time to dig deeper and understand what happened. I hope that is OK. 🙏

@lightbody, Yes, I apologize for any inconvenience caused. Upon further review, I realized that the filter logic could benefit from refinements. I've been testing a version locally that seems effective in my current setup. However, I'm concerned that it might not scale well with a larger number of accounts or meetings. Hence, I haven't submitted a new PR yet.
One significant challenge I encountered was distinguishing whether an event has been synced from another. I didn't filter all the reclaimManaged && personalSync events primarily because external calendars linked to Reclaim (and subsequently synced to a Reclaim-linked Google calendar) appear only once in the events endpoint.
@lightbody, if you have insights or suggestions about which properties or endpoints might be relevant for this issue, I'd greatly appreciate it. I'm eager to collaborate and provide any assistance.
This is my current fulter logic used locally
/**
 * Filter out events that are synced, managed by Reclaim and part of multiple calendars
 * @param events
 * @returns
 */
export const filterMultipleOutDuplicateEvents = <Events extends ApiResponseEvents | undefined>(events: Events) => {
  if (!events) return events;

  // Create a map to store counts of each start-end combination
  const eventTimeCounts = new Map<string, number>();

  events.forEach((event) => {
    const key = `${event.eventStart}-${event.eventEnd}`;
    eventTimeCounts.set(key, (eventTimeCounts.get(key) ?? 0) + 1);
  });

  const res = events.filter((event) => {
    const key = `${event.eventStart}-${event.eventEnd}`;
    const count = eventTimeCounts.get(key) as number;

    // If the event has both personalSync and reclaimManaged and there's more than one event with the same start-end combination, we don't include it
    return !(event.personalSync && event.reclaimManaged && count > 1);
  });

  console.log(res.map((event) => [event.title, event.color]));

  return res;
};

Is this ready to merge or do we want to change the author in package.json to another Raycast handle?

if it's possible to change it, then yes, we'd like to have the author changed to either myself or to @lightbody (CEO of Reclaim.ai).


Portainer does not respect '--no-healthcheck' in Command for Node-Red container (but Docker itself does)

Bug description
Portainer does not respect '--no-healthcheck' option when running a Node-Red container. When I execute the following via SSH, the '--no-healthcheck' option works fine, the container gets listed in Portainer and runs without healtcheck (status "running")."
 docker run -it -p 1880:1880 --no-healthcheck -v /srv/dev-disk-by-uuid-95f9376f-4b40-4f41-a85b-615712219901/data/nodered:/data --name node-red nodered/node-red
When the same container is stopped and re-started (via Portainer), the no-healthcheck is not respected any more and the container gets status "healthy", which means healtcheck is enabled.
I put  '--no-healthcheck' into the field called Command (Override) in section "Command & logging". It does not matter if I create a new container or edit via "Duplicate/Edit".
So, in other words, when running a container via SSH and "docker run..." everything works as expected (healthcheck disabled) but if running the same container (or restarting) in Portainer, healthcheck is not getting disabled. This leads to a conclusion that Portainer manages somehow '--no-healthcheck' option wrong way.
I want to use '--no-healtcheck' option as Node-Red container has healtcheck enabled by default and it causes excessive logging (Syslog):
May 23 13:41:14 nebulonserver systemd[1]: run-docker-runtime\x2drunc-moby-00d4120caead6372631b47f1e0d98c2c0c138679fb21f63b15f930a60e58b4e3-runc.iPwfl5.mount: Succeeded. May 23 13:41:45 nebulonserver systemd[968]: run-docker-runtime\x2drunc-moby-00d4120caead6372631b47f1e0d98c2c0c138679fb21f63b15f930a60e58b4e3-runc.CBunCG.mount: Succeeded. May 23 13:41:45 nebulonserver systemd[1]: run-docker-runtime\x2drunc-moby-00d4120caead6372631b47f1e0d98c2c0c138679fb21f63b15f930a60e58b4e3-runc.CBunCG.mount: Succeeded. May 23 13:42:16 nebulonserver systemd[968]: run-docker-runtime\x2drunc-moby-00d4120caead6372631b47f1e0d98c2c0c138679fb21f63b15f930a60e58b4e3-runc.86GJsi.mount: Succeeded. May 23 13:42:46 nebulonserver systemd[968]: run-docker-runtime\x2drunc-moby-00d4120caead6372631b47f1e0d98c2c0c138679fb21f63b15f930a60e58b4e3-runc.c1IvKK.mount: Succeeded. May 23 13:43:17 nebulonserver systemd[968]: run-docker-runtime\x2drunc-moby-00d4120caead6372631b47f1e0d98c2c0c138679fb21f63b15f930a60e58b4e3-runc.YILx1f.mount: Succeeded. May 23 13:43:47 nebulonserver systemd[1]: run-docker-runtime\x2drunc-moby-00d4120caead6372631b47f1e0d98c2c0c138679fb21f63b15f930a60e58b4e3-runc.26eNHT.mount: Succeeded. May 23 13:43:47 nebulonserver systemd[968]: run-docker-runtime\x2drunc-moby-00d4120caead6372631b47f1e0d98c2c0c138679fb21f63b15f930a60e58b4e3-runc.26eNHT.mount: Succeeded.
Expected behavior
I would like Portainer to respect and add properly 'no-healthcheck' option to a run container.
Technical details:

Raspberry Pi 4 4GB
OMV 5.6.7.-1
Docker version 5:20.10.6~3-0~debian-buster installed via OMV
Portainer 2.11 installed via OMV
Linux 5.10.17-v7I+ (Raspberry Pi OS - "2021-03-04-raspios-buster-armhf.zip")

Hi
its a known limitation with UI , there is a feature request for the same, please follow this
https://github.com/portainer/portainer/issues/827

I suspect this might not be complicated to implement - but will touch the entire UI->backend stack.


moodle: fix typo

Motivation for this change
Fix typo introduced in b6eca9a2afa143bcbee3b4ae1bdc13993bc71784
cc @Lassulus @Kloenk @aanderse
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Sorry, missed that. Lgtm


Content: Text component

Including use cases, how to use guidance, accessibility guidelines and more.
Acceptance Criteria:

[x] Component Kickoff
[x] Component review
[x] Sync with Dev and Design
[x] Research and consulting other design libraries
[x] Component ID (name, definition, example image
[x] Use case (when to use guidance & related components)
[x] How to use guidance
[x] Accessibility guidelines
[x] Inclusive language
[x] Descriptive text for images
[ ] Translation & Localization
[ ] Bilingual review – content design
[ ] Ensure consistency: ordering, terminology, high-level voice/tone/style
[ ] Final review

First English review done, made edits and left comments
https://docs.google.com/document/d/19mc3lGeW92Co9VdjnbV4g9U9kYh9P0_Lw5-s08nMDD8/edit

Reviewed, made comments and reworked some of the text. Added some content guidance.

Working on two guidance sections on what to include in text component content. Pulled in some content from error message component design guidance. Made a Slack post in #content-designers Slack channel to ask for contributions from the content design community of practice. https://gcdigital.slack.com/archives/CJGDB0TQV/p1699024935219149

Private Zenhub Image

Document ready for translation @nchoquet

Translation's back. @CelesteCote doing the bilingual review.

Reviewing now. And dropping the link to the French version here for handy reference: https://docs.google.com/document/d/1CXsORRPK7ELQoktblhpI_21Ubzdg4p_AEC8WkLwxdgg/edit

Reviewing now. And dropping the link to the French version here for handy reference: https://docs.google.com/document/d/1CXsORRPK7ELQoktblhpI_21Ubzdg4p_AEC8WkLwxdgg/edit

Reviewed and responded to comments on the English text

Guidance added to documentation site in PR #270.


Serialization round-trip fails with empty Vec

When serializing a struct containing an empty Vec, the Vec is completely dropped.
But when de-serializing the struct, the Vec is required.
This breaks the serialization round-trip.
My expectation is that the de-serializing should work, even if the Vec is not present.
Here is an example:
#[derive(Debug, PartialEq, Eq, serde::Serialize, serde::Deserialize)]
struct Data {
    values: Vec<String>,
}

#[test]
fn serialization_roundtrip() {
    let data = Data { values: Vec::new() };

    let serialized = serde_qs::to_string(&data).unwrap();

    dbg!(&serialized);

    let deserialized = serde_qs::from_str::<Data>(&serialized).unwrap();

    assert_eq!(deserialized, data);
}

Hey @dfaust!
Thanks for opening an issue. You raise an interesting problem. I'm struggling to come up with a way that we could solve this on the serde_qs side. I don't think there's any way for us to represent an empty vector.
One thing you could maybe do as a workaround it:
#[test]
fn serialization_roundtrip() {
    #[derive(Debug, PartialEq, Eq, serde::Serialize, serde::Deserialize)]
    struct Data {
        #[serde(default)] // <<<<< use this
        values: Vec<String>,
    }

    let data = Data { values: Vec::new() };
    let serialized = serde_qs::to_string(&data).unwrap();

    dbg!(&serialized);
    let deserialized = serde_qs::from_str::<Data>(&serialized).unwrap();
    assert_eq!(deserialized, data);
}

Do you think that would be possible?

Thanks @samscott89 for the quick reply!

Do you think that would be possible?

Sure. That's an easy enough work-around. :+1:
I still hope that you can find a solution though, since this behavior is quite surprising.
Anyway, I still love your crate. Being able to serialize Vecs as URLs is fantastic!

Issue also occurs with an empty HashSet


how to use Middleware groups?

My route file was like this
Route::group(['middleware' => ['web']], function () {

    Route::get('/', function () {
        return view('welcome');
    });

Now I have created PublicRouter class,
How can i define middleware group 'web' for my public routes?
Thanks

Hi @coding-sunshine. You can do this as you normally would :) Just place the following in the map method of your router:
$this->router->group(['middleware' => ['web']], function () {

    $this->router->get('/', function () {
        return view('welcome');
    });

});

In any case, you can opt to use the Route facade instead of $this->router if you prefer it that way. So your given example should work just fine.

Thanks for the reply.
Actually I guessed it initially and tried the same but it gives me following error
ReflectionException in Container.php line 738: Class web does not exist

    in Container.php line 738
    at ReflectionClass->__construct('web') in Container.php line 738
    at Container->build('web', array()) in Container.php line 633
    at Container->make('web', array()) in Application.php line 697
    at Application->make('web') in Pipeline.php line 126
    ...

Am I doing something wrong?

@coding-sunshine I've identified the issue at hand. Will fix asap :)

@coding-sunshine Can you confirm you're running Laravel 5.2 and have the following in your App\Http\Kernel.php file?
    /**
     * The application's route middleware groups.
     *
     * @var array
     */
    protected $middlewareGroups = [
        'web' => [
            \App\Http\Middleware\EncryptCookies::class,
            \Illuminate\Cookie\Middleware\AddQueuedCookiesToResponse::class,
            \Illuminate\Session\Middleware\StartSession::class,
            \Illuminate\View\Middleware\ShareErrorsFromSession::class,
            \App\Http\Middleware\VerifyCsrfToken::class,
        ],

        'api' => [
            'throttle:60,1',
        ],
    ];

    /**
     * The application's route middleware.
     *
     * These middleware may be assigned to groups or used individually.
     *
     * @var array
     */
    protected $routeMiddleware = [
        'auth'       => \App\Http\Middleware\Authenticate::class,
        'auth.basic' => \Illuminate\Auth\Middleware\AuthenticateWithBasicAuth::class,
        'can'        => \Illuminate\Foundation\Http\Middleware\Authorize::class,
        'guest'      => \App\Http\Middleware\RedirectIfAuthenticated::class,
        'throttle'   => \Illuminate\Routing\Middleware\ThrottleRequests::class,
    ];

Had the same issue, but was testing it with Laravel 5.1, which of course doesn't have the middleware groups and was throwing the same error.

I am using Laravel 5.2.31
and my Kernel.php file looks as follows:
<?php

namespace App\Http;

//use Illuminate\Foundation\Http\Kernel as HttpKernel;
use SebastiaanLuca\Router\Kernel as HttpKernel;
use App\Http\Routers\PublicRouter;

class Kernel extends HttpKernel
{
    /**
     * The application's global HTTP middleware stack.
     *
     * These middleware are run during every request to your application.
     *
     * @var array
     */
    protected $middleware = [
        \Illuminate\Foundation\Http\Middleware\CheckForMaintenanceMode::class,
        \App\Http\Middleware\MyRobotsMiddleware::class,
    ];

    /**
     * The application's route middleware groups.
     *
     * @var array
     */
    protected $middlewareGroups = [
        'web' => [
            \App\Http\Middleware\EncryptCookies::class,
            \Illuminate\Cookie\Middleware\AddQueuedCookiesToResponse::class,
            \Illuminate\Session\Middleware\StartSession::class,
            \Illuminate\View\Middleware\ShareErrorsFromSession::class,
            \App\Http\Middleware\VerifyCsrfToken::class,
        ],

        'api' => [
            'throttle:60,1',
        ],
    ];

    /**
     * The application's route middleware.
     *
     * These middleware may be assigned to groups or used individually.
     *
     * @var array
     */
    protected $routeMiddleware = [
        'auth' => \App\Http\Middleware\Authenticate::class,
        'allowedto' => \Spatie\Authorize\Middleware\Authorize::class,
        'auth.basic' => \Illuminate\Auth\Middleware\AuthenticateWithBasicAuth::class,
        'can' => \Illuminate\Foundation\Http\Middleware\Authorize::class,
        'guest' => \App\Http\Middleware\RedirectIfAuthenticated::class,
        'throttle' => \Illuminate\Routing\Middleware\ThrottleRequests::class,
    ];

    /**
     * The routers to automatically map.
     *
     * @var array
     */
    protected $routers = [
        PublicRouter::class,
    ];
}

@coding-sunshine Pushed a fix and extensively tested it. Run composer update and you'll be good to go. Feedback very much appreciated :)
https://github.com/sebastiaanluca/laravel-router/releases

Yup,
this worked fine.
Thanks very much for help.


Fix: Crossfade animation causes other animations to have hidden elements

What does the pull request do?
Fix for CrossFade animation overwriting opacity causing elements to not show if you change the transition style (eg. PageSlide or none)
What is the current behavior?
When switching transition types on a control (EG. TransitioningContentControl) the opacity change is not reversed. This causes content to not show on other transition types.
This can be tested using the Catalog samples TransitioningContentControl.
Use the cross fade transition and move to another element in the control, then change to none and when transitioning some elements are hidden.
What is the updated/expected behavior with this PR?
Elements will maintain their existing opacity after a transition using CrossFade.
This can be tested using the Catalog samples TransitioningContentControl.
Use the cross fade transition and move to another element in the control, then change to none and when transitioning, elements should show as expected.
How was the solution implemented (if it's not obvious)?
Remove the opacity changes on the to/from Visual components and supporting code as the transition keyframes will take care of the opacity.
Fixed issues
Fixes #14054

@dotnet-policy-service agree


Allow structs in g/->dataset

[x] I have read through the quick start and installation sections of the README.

Info

Info
Value

Geni Version
0.0.38

Problem / Steps to reproduce

Cannot use structs or list of structs in g/->dataset, example:
(-> (g/->dataset [{:id "bob"   :skills ["javascript" "clojure"] :preference {:remote true}  :experience [{:name "CompanyA" :years 2}
                                                                                {:name "CompanyB" :years 1}]}
                    {:id "alice" :skills ["python" "c++"]         :experience []}])
      g/show)

leads to
; Execution error (ClassCastException) at org.apache.spark.sql.catalyst.expressions.CastBase/buildCast (Cast.scala:295).
; class clojure.lang.Keyword cannot be cast to class [B (clojure.lang.Keyword is in unnamed module of loader 'app'; [B is in module java.base of loader 'bootstrap')

Running g/print-schema right before g/show above shows incorrect schema:
root
 |-- id: string (nullable = true)
 |-- skills: array (nullable = true)
 |    |-- element: string (containsNull = true)
 |-- preference: array (nullable = true)
 |    |-- element: array (containsNull = true)
 |    |    |-- element: binary (containsNull = true)
 |-- experience: array (nullable = true)
 |    |-- element: array (containsNull = true)
 |    |    |-- element: array (containsNull = true)
 |    |    |    |-- element: binary (containsNull = true)

Expected Result
g/show should output:
+-----+---------------------+----------+------------------------------+
|id   |skills               |preference|experience                    |
+-----+---------------------+----------+------------------------------+
|bob  |[javascript, clojure]|{true}    |[{CompanyA, 2}, {CompanyB, 1}]|
|alice|[python, c++]        |null      |[]                            |
+-----+---------------------+----------+------------------------------+

g/print-schema should output:
root
 |-- id: string (nullable = true)
 |-- skills: array (nullable = true)
 |    |-- element: string (containsNull = true)
 |-- preference: struct (nullable = true)
 |    |-- remote: boolean (nullable = true)
 |-- experience: array (nullable = true)
 |    |-- element: struct (containsNull = true)
 |    |    |-- name: string (nullable = true)
 |    |    |-- years: long (nullable = true)

Proposed Solution

Check for map? in infer-spark-type at https://github.com/zero-one-group/geni/blob/123aec55351adad893907ea4284e5b2e0ce43f45/src/clojure/zero_one/geni/core/dataset_creation.clj#L130
If any maps exist in table, transform to array of values https://github.com/zero-one-group/geni/blob/123aec55351adad893907ea4284e5b2e0ce43f45/src/clojure/zero_one/geni/core/dataset_creation.clj#L165

I have managed to implement the solution locally, just need to add some tests.

Hi @WaqasAliAbbasi, thanks for this! If I recall correctly, I did not think about this case. I'll check out your PR!
Are you using Geni as a dependency? Will you need a new release for these changes?

Hi @WaqasAliAbbasi, thanks for this! If I recall correctly, I did not think about this case. I'll check out your PR!
Are you using Geni as a dependency? Will you need a new release for these changes?

Thanks for looking into the PR, will update it soon.
Yes we use geni heavily in our team (its such a neat library, thanks for your work 😃) Will appreciate it if you push a release once this change is in.

Thank you very much for the kind words. I'm really happy to hear that!
Please let me know once you're happy with the PR, I'll merge it and create a release!


.net build failed

Problem encountered on https://dotnet.microsoft.com/learn/dotnet/hello-world-tutorial/edit
Operating System: windows
Provide details about the problem you are experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.

Hi @Hazeez-ur-Rehman! Can you share what your code looks like after you've modified it? It seems it had some issues in the modified code since it ran ok the first time.

Closing due to the lack of response.


Implement functionality from #18 for resources

Implementing a similar approach to #18 for resources would improve performance and make the API more consistent

[x] Implement new API for resources similar to #18
[x] optimize performance with better use of RwLocks

ecf2490


btContentLocal table seems to be installing twice.

The install fails because btContentLocal is trying to be created in MySQL and it was already installed in the setup script.
Install concrete5
Version 5.7.0a3
An exception occurred while executing 'CREATE TABLE btContentLocal (bID INT UNSIGNED NOT NULL, content LONGTEXT DEFAULT NULL, PRIMARY KEY(bID)) DEFAULT CHARACTER SET utf8 COLLATE utf8_unicode_ci ENGINE = InnoDB': SQLSTATE[42S01]: Base table or view already exists: 1050 Table 'btcontentlocal' already exists

having this issue with <IP_ADDRESS>
multiple tries of running the installation
failing randomly at one of those tables so far:

UserBannedIPs
atNumber
atSocialLinks

here is one of the stacktraces
An exception occurred while executing 'CREATE TABLE UserBannedIPs (id INT UNSIGNED AUTO_INCREMENT NOT NULL, ipFrom TINYBLOB DEFAULT NULL, ipTo TINYBLOB DEFAULT NULL, banCode TINYINT(1) DEFAULT '1' NOT NULL, expires DATETIME DEFAULT '0000-00-00 00:00:00' NOT NULL, isManual TINYINT(1) DEFAULT '0' NOT NULL, PRIMARY KEY(id)) DEFAULT CHARACTER SET utf8 COLLATE utf8_unicode_ci ENGINE = InnoDB':

SQLSTATE[42S01]: Base table or view already exists: 1050 Table 'UserBannedIPs' already exists.

Trace:
#0 /srv/www/vhosts/example.com/doc/concrete5/concrete/vendor/doctrine/dbal/lib/Doctrine/DBAL/Connection.php(811): Doctrine\DBAL\DBALException::driverExceptionDuringQuery(Object(PDOException), 'CREATE TABLE Us...')
#1 [internal function]: Doctrine\DBAL\Connection->query('CREATE TABLE Us...')
#2 /srv/www/vhosts/example.com/doc/concrete5/concrete/src/Database/Connection/Connection.php(78): call_user_func_array('parent::query', Array)
#3 /srv/www/vhosts/example.com/doc/concrete5/concrete/src/Package/Package.php(217): Concrete\Core\Database\Connection\Connection->query('CREATE TABLE Us...')
#4 /srv/www/vhosts/example.com/doc/concrete5/concrete/src/Package/StartingPointPackage.php(190): Concrete\Core\Package\Package::installDB('/srv/www/vhosts...')
#5 [internal function]: Concrete\Core\Package\StartingPointPackage->install_database()
#6 /srv/www/vhosts/example.com/doc/concrete5/concrete/controllers/install.php(286): call_user_func(Array)
#7 [internal function]: Concrete\Controller\Install->run_routine('elemental_full', 'install_databas...')
#8 /srv/www/vhosts/example.com/doc/concrete5/concrete/src/Controller/AbstractController.php(156): call_user_func_array(Array, Array)
#9 /srv/www/vhosts/example.com/doc/concrete5/concrete/src/Routing/ControllerRouteCallback.php(25): Concrete\Core\Controller\AbstractController->runAction('run_routine', Array)
#10 /srv/www/vhosts/example.com/doc/concrete5/concrete/src/Routing/Router.php(59): Concrete\Core\Routing\ControllerRouteCallback->execute(Object(Concrete\Core\Http\Request), Object(Concrete\Core\Routing\Route), Array)
#11 /srv/www/vhosts/example.com/doc/concrete5/concrete/src/Support/Facade/Facade.php(116): Concrete\Core\Routing\Router->execute(Object(Concrete\Core\Routing\Route), Array)
#12 /srv/www/vhosts/example.com/doc/concrete5/concrete/src/Application/Application.php(342): Concrete\Core\Support\Facade\Facade::__callStatic('execute', Array)
#13 /srv/www/vhosts/example.com/doc/concrete5/concrete/src/Application/Application.php(342): Concrete\Core\Support\Facade\Route::execute(Object(Concrete\Core\Routing\Route), Array)
#14 /srv/www/vhosts/example.com/doc/concrete5/concrete/bootstrap/start.php(196): Concrete\Core\Application\Application->dispatch(Object(Concrete\Core\Http\Request))
#15 /srv/www/vhosts/example.com/doc/concrete5/concrete/dispatcher.php(36): require('/srv/www/vhosts...')
#16 /srv/www/vhosts/example.com/doc/concrete5/index.php(2): require('/srv/www/vhosts...')
#17 {main}

currently it is not possible to test lower_case_table_names=0

for 5.7.4RC2 i get this message. but even after that exception the database grows to 208 tables.
There are already 128 tables in this database. concrete5 must be installed in an empty database..

Trace:
#0 [internal function]: Concrete\Core\Package\StartingPointPackage->install_database()
#1 /srv/www/vhosts/example.com/doc/concrete5/concrete/controllers/install.php(264): call_user_func(Array)
#2 [internal function]: Concrete\Controller\Install->run_routine('elemental_full', 'install_databas...')
#3 /srv/www/vhosts/example.com/doc/concrete5/concrete/src/Controller/AbstractController.php(156): call_user_func_array(Array, Array)
#4 /srv/www/vhosts/example.com/doc/concrete5/concrete/src/Routing/ControllerRouteCallback.php(25): Concrete\Core\Controller\AbstractController->runAction('run_routine', Array)
#5 /srv/www/vhosts/example.com/doc/concrete5/concrete/src/Routing/Router.php(59): Concrete\Core\Routing\ControllerRouteCallback->execute(Object(Concrete\Core\Http\Request), Object(Concrete\Core\Routing\Route), Array)
#6 /srv/www/vhosts/example.com/doc/concrete5/concrete/src/Support/Facade/Facade.php(119): Concrete\Core\Routing\Router->execute(Object(Concrete\Core\Routing\Route), Array)
#7 /srv/www/vhosts/example.com/doc/concrete5/concrete/src/Application/Application.php(368): Concrete\Core\Support\Facade\Facade::__callStatic('execute', Array)
#8 /srv/www/vhosts/example.com/doc/concrete5/concrete/src/Application/Application.php(368): Concrete\Core\Support\Facade\Route::execute(Object(Concrete\Core\Routing\Route), Array)
#9 /srv/www/vhosts/example.com/doc/concrete5/concrete/bootstrap/start.php(217): Concrete\Core\Application\Application->dispatch(Object(Concrete\Core\Http\Request))
#10 /srv/www/vhosts/example.com/doc/concrete5/concrete/dispatcher.php(36): require('/srv/www/vhosts...')
#11 /srv/www/vhosts/example.com/doc/concrete5/index.php(2): require('/srv/www/vhosts...')
#12 {main}

Can you provide more information about the environment?

@Mnkras
Webserver:
OS: openSUSE 13.1
Apache: 2.4.6
PHP: 5.4.20
DB Server:
OS: openSUSE 13.1
MySQL: 5.6.12
we have mutltiple 5.6.x installations running on this setup
do you need more info?

any news?
i just found that according to MySQL Doc the default value of lower_case_table_names on unix systems is already 0.

On Unix, the default value of lower_case_table_names is 0

We have a Project where we need a new installation of concrete5 and do not want to use a 5.6 version as there appears to be no way to upgrade to 5.7 later.

I just tried to install concrete5 5.7.4RC2 on a clean openSUSE 13.1, with the default packages:

PHP 5.4.20
Apache 2.4.6
MySQL 5.5.33

concrete5 installed flawlessly... I don't know what can be wrong in your installation.
Maybe the output of MySQL SHOW VARIABLES; and php -i could help...

here you can find both.

I setup a system with the exact components that you have (MySQL 5.6.12, PHP 5.4.20 and Apache 2.4.6), and I still have no problems.
I compared the configuration files and the differences is that you have xcache and I have mbstring, but those shouldn't affect the installation IMHO.
At this point I think that the difference is in the MySQL setup. In my test enviro MySQL is on the same machine as Apache, and I access it with root. From you write it seems you're using two different servers.
Does the mysql user that access the DB has limited access? Maybe it's a permission problem. With the mysql user that you use, you can run the query SHOW FULL TABLES WHERE Table_type = 'BASE TABLE'?

yeah. we have multiple servers where in this case one is the db and one the apache.
i can execute the query and get all tables i also see with show tables; but with another column that is BASE TABLE for each row.
mysql> show grants;
+--------------------------------------------------------------------------------------------------------+
| Grants for *user*@%                                                                                |
+--------------------------------------------------------------------------------------------------------+
| GRANT USAGE ON *.* TO '*user*'@'%' IDENTIFIED BY PASSWORD '**password*' |
| GRANT ALL PRIVILEGES ON `*c5database*`.* TO '*user*'@'%'                                           |
+--------------------------------------------------------------------------------------------------------+

I ran out of ideas. Maybe taking a look at the php error log could reveal what's going wrong...

our error log does not show anything related to concrete5 during installation. only the installer itself shows the errors posted above. but for some reason, most of the time i have diffrent tables that already exist (for <IP_ADDRESS>) but always i have 203 tables that got created in the database.
here is a snipped of the access log right. starting with the button click on the page where i configure the database

whops hit ctrl-return....
*my-ip* - - [07/May/2015:10:34:41 +0200] "POST /concrete5/index.php/install/configure HTTP/1.1" 302 428 "http://www.example.com/concrete5/index.php/install/configure" "Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/42.0.2311.135 Safari/537.36"
*my-ip* - - [07/May/2015:10:34:42 +0200] "GET /concrete5/index.php HTTP/1.1" 302 460 "http://www.example.com/concrete5/index.php/install/configure" "Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/42.0.2311.135 Safari/537.36"
*my-ip* - - [07/May/2015:10:34:42 +0200] "GET /concrete5/index.php/install HTTP/1.1" 200 19591 "http://www.example.com/concrete5/index.php/install/configure" "Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/42.0.2311.135 Safari/537.36"
*my-ip* - - [07/May/2015:10:34:43 +0200] "GET /concrete5/index.php/install/run_routine/elemental_full/make_directories HTTP/1.1" 200 15 "http://www.example.com/concrete5/index.php/install" "Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/42.0.2311.135 Safari/537.36"
*my-ip* - - [07/May/2015:10:36:13 +0200] "GET /concrete5/index.php/install/run_routine/elemental_full/install_database HTTP/1.1" 200 3920 "http://www.example.com/concrete5/index.php/install" "Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/42.0.2311.135 Safari/537.36"
*my-ip* - - [07/May/2015:10:34:43 +0200] "GET /concrete5/index.php/install/run_routine/elemental_full/install_database HTTP/1.1" 200 15 "http://www.example.com/concrete5/index.php/install" "Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/42.0.2311.135 Safari/537.36"

btw the 203 tables are always the same, as i can drop them everytime with the same drop table command (containing all table names).
so somehow the script seems to always have a diffrent order for table creation.
also the error is shown with the first request do /install_database and when i check my chrome network tab the second request is not even shown.

@mlocati could you dump a clean installation database for me to test?

Sure! Here it is.
Could you tell me when you don't need it anymore? So, I'll delete that file. Thanks

sorry i had no time to work on this. i downloaded the file.
not sure when i get the time to test if it works. if i can import the dump how can i login? so that we maybe can use this as a start for our installation.
Thanks! :)

hey @mlocati i just tried to install the <IP_ADDRESS> but again errors while createing the database.
this time:
There are already 167 tables in this database. concrete5 must be installed in an empty database..

Trace:
#0 [internal function]: Concrete\Core\Package\StartingPointPackage->install_database()
#1 /srv/www/vhosts/example.com/doc/concrete5/concrete/controllers/install.php(264): call_user_func(Array)
#2 [internal function]: Concrete\Controller\Install->run_routine('elemental_full', 'install_databas...')
#3 /srv/www/vhosts/example.com/doc/concrete5/concrete/src/Controller/AbstractController.php(156): call_user_func_array(Array, Array)
#4 /srv/www/vhosts/example.com/doc/concrete5/concrete/src/Routing/ControllerRouteCallback.php(25): Concrete\Core\Controller\AbstractController->runAction('run_routine', Array)
#5 /srv/www/vhosts/example.com/doc/concrete5/concrete/src/Routing/Router.php(59): Concrete\Core\Routing\ControllerRouteCallback->execute(Object(Concrete\Core\Http\Request), Object(Concrete\Core\Routing\Route), Array)
#6 /srv/www/vhosts/example.com/doc/concrete5/concrete/src/Support/Facade/Facade.php(119): Concrete\Core\Routing\Router->execute(Object(Concrete\Core\Routing\Route), Array)
#7 /srv/www/vhosts/example.com/doc/concrete5/concrete/src/Application/Application.php(366): Concrete\Core\Support\Facade\Facade::__callStatic('execute', Array)
#8 /srv/www/vhosts/example.com/doc/concrete5/concrete/src/Application/Application.php(366): Concrete\Core\Support\Facade\Route::execute(Object(Concrete\Core\Routing\Route), Array)
#9 /srv/www/vhosts/example.com/doc/concrete5/concrete/bootstrap/start.php(217): Concrete\Core\Application\Application->dispatch(Object(Concrete\Core\Http\Request))
#10 /srv/www/vhosts/example.com/doc/concrete5/concrete/dispatcher.php(36): require('/srv/www/vhosts...')
#11 /srv/www/vhosts/example.com/doc/concrete5/index.php(2): require('/srv/www/vhosts...')
#12 {main}

and before you ask... the database was empty when i started the installation.
then i wanted to try your sql dump but i just realized that i lost the old zip files (5.7.3 & 5.7.4-rc) so that i cant test your database. do you mind to do another dump with <IP_ADDRESS>?
Thanks! :)

There are already 167 tables in this database.

Btw: after that installation run/try, the database contained 204 tables.

The link above is still valid

after i had to fix some smaller issues in the dump (remove a linebreak from a timestamp & fix an invalid default value for msPluralCases) i managed to import the database but now i realized, that i need a confg file. i tried to copy the config/site.php from one of our c5.6 installations and changed the values but no success. is it possible that you could create a dummy config that i could use?

I don't know if installing such files solves your problem. You should really check your configuration, there's something really wrong in your server.
BTW, here's the files you need:
Contents of application/config/database.php
<?php

return array(
    'default-connection' => 'concrete',
    'connections' => array(
        'concrete' => array(
            'driver' => 'c5_pdo_mysql',
            'server' => 'YOUR-DATABASE-SERVER',
            'database' => 'YOUR-DATABASE-NAME',
            'username' => 'YOUR-DATABASE-USERNAME',
            'password' => 'YOUR-DATABASE-PASSWORD',
            'charset' => 'utf8'
        )
    )
);

Contents of application/config/generated_overrides/concrete.php
<?php

return array(
    'site' => 'NAME_OF_YOUR_SITE',
    'version_installed' => 'YOUR-CONCRETE5-VERSION'
);

nope. those files do not help yet.
Doctrine\Common\Proxy\AbstractProxyFactory::getProxyDefinition(): Failed opening required '/srv/www/vhosts/example.com/doc/concrete<IP_ADDRESS>/application/config/doctrine/proxies/__CG__ConcreteCoreFileStorageLocationStorageLocation.php' (include_path='/srv/www/vhosts/example.com/doc/concrete<IP_ADDRESS>/concrete/vendor:.:/usr/share/php5:/usr/share/php5/PEAR')

i just tried to install concrete5 from outside of our company network and it worked...
it seems that our firewall/proxy setup was causing that issue somehow... (we have multiple level of firewalls & proxies)
thank you for your time and help...

Glad to see you solved this issue. I'm just curious about what was blocking what

I wish I'd know what exactly was blocking... For some reason it did not work when using our proxies/firwalls... i tested with a PC that is not in our company network and it just installed fine.


Possibility to change an order of elements in URL hints and set the limit of URL hints

Welcome,
Is it possible to add an option to change an order of elements in URL hints which are shown when I type addresses in and to set the limit of the URL hints from searching engine?
I think that a possibility to set those preferences might be usable for some people (including me).
This way one might avoid moving down the (sometimes quite long) list of the search engine URL hints to get to some matching bookmark or history entries.
Best regards,
mp107

+1


Firebase connect and disconnet

User use below call

fcm.connect()

Connect Firebase and get token
Add event Handler to push notification

fcm.disconnect()

Disconnect Firebase and delete token

@daehyunjimmy Your patch looks good to me but please split this into smaller patches for easier review.


foreach() in index.php

When visiting my nsgsb/stats/ page, just under "Top Referers" I get:
Warning: Invalid argument supplied for foreach() in /path/to/index.php on line 195

@spiralofhope Thanks for reporting that.
Looks like that warning slipped through because I haven't been running the script with PHP errors enabled lately.
It led me to a typo that was causing top referrers on the slug stats page to be empty and leading to the error you're seeing. git pull the latest changes and it should be fixed 👍

The latest git pull resolves the issue for me.
Thank you very much.


Support for URL-safe decoding

rustc-serialize supports three types of encoding:
MIME: Configuration for RFC 2045 MIME base64 encoding
STANDARD: Configuration for RFC 4648 standard base64 encoding
URL_SAFE: Configuration for RFC 4648 base64url encoding
In particular, I have a use case for URL-safe encoding/decoding.

Adding a mode for url-safe now, will likely get to MIME later this week and will close when that's complete. Thank you!

later this week
:open_mouth:

mime added with #19


Nuxt - Global middleware override and sort issue with layer

Environment

Operating System: Linux
Node Version:     v18.18.0
Nuxt Version:     3.7.4
CLI Version:      3.9.0
Nitro Version:    2.6.3
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      extends
Runtime Modules:  -
Build Modules:    -

Reproduction
https://stackblitz.com/edit/nuxt-starter-4ryqhc?file=pages%2Findex.vue
Describe the bug
Global middleware order and override seems to be not in the logic layer way.
base layer middleware shoud be overriden by project middleware and sorting should be alpha of all layers.
From the repro
Console should show
00-middleware base
05-middleware project
10-middleware project
20-middleware base
But show
00-middleware base
10-middleware bas >
20-middleware base
05-middleware project
For me 10-middleware base shoud be overriden by 10-middleware project
AND 05-middleware project should be just after 00-middleware base
I have made a module to correct the middleware sort (juste rename base__modules foledr to base\module )
But can't find how to solve the not overriden middleware.
Additional context
No response
Logs
No response

The ordering of middleware is intentional - see https://github.com/nuxt/nuxt/pull/22889#discussion_r1311504048 and https://github.com/nuxt/nuxt/pull/22925. You can adjust the middleware ordering as you are doing in a module or hook.
But indeed the global middleware should be overrideable; this is a bug that should be fixed.


InMemory SearchExtension bug

There is a bug in the SearchExtension class used by the InMemory SearchEvaluator.
When the input string contains the requested pattern, BUT the first character of said pattern is used in the power of 2, the result is false.
Note the amount of 'a' characters in the screenshot

Hi @Jordy84,
We're aware that the in-memory implementation might fail in certain edge cases. The Like operator is quite complex, and it lacks standardization across different database providers. So, it's unlikely that we'll ever have 1:1 match in functionality.
But, your case contains quite basic usage, and it's an obvious bug. We must fix it. Thanks for reporting it.

Ok, this turned out to be trickier than I anticipated.
I implemented it using regex (after many trials and errors). It will not be as performant as the previous version, but at least it behaves correctly.

Thank you for the quick fix!


Show "Connecting…" banner (or equivalent) when server data is stale

When we lose our real-time stream of events from the server, either because of network failures or because the server has expired the event queue and we need to request a new one:

#184
#185

we continue to show the latest server data we had, but it may now be stale. We should communicate to the user that that's what's happening.
In the existing zulip-mobile app, we do this with a banner that reads "Connecting…" and appears just under the app bar.
In this app, we could do the same thing, or we could find a different design. One downside of the "Connecting…" banner is that it occupies some vertical space, which means the rest of the content needs to move down to make room (or else let its top portion be obscured by the banner). Nevertheless it might be the best available option.
Even if we use the same design concept, we'll want to pin down the details: the text style, the layout, the banner's background, the specifics of how it animates in and out.
(In zulip-mobile the details are in src/common/LoadingBanner.js; there's no animation. But the conceptually similar "No Internet connection" banner, found in src/boot/OfflineNoticeProvider.js, does have an animation… on iOS only, because React Native's implementation breaks on Android. Here on Flutter, we should have no such trouble implementing whatever animation we choose.)

Bumping this in priority because even after #185 and #184, we're still sometimes showing the symptom of lacking an event queue, even when the device has a fine internet connection:
https://chat.zulip.org/#narrow/stream/48-mobile/topic/Only.20.22all.20messages.22.20updating/near/1754151
(I guess Alex didn't explicitly say the last part, but I definitely had a fine connection when I observed similar symptoms the other day.)
And as Alex said there:

silently not having updates means I cannot trust the app. Having some user feedback seems very important, here.

Maybe something like a toast?
Or a thin, linear indeterminate progress indicator at the bottom of the app bar (LinearProgressIndicator), like we do when logging you in? (It disappears quickly in this video, but it's there.)
https://github.com/zulip/zulip-flutter/assets/22248748/0b63a678-37ab-49c3-8365-ca80579cb1ab

a thin, linear indeterminate progress indicator at the bottom of the app bar (LinearProgressIndicator), like we do when logging you in?

Yeah, that sounds like the easiest thing to do that remains pretty clean-looking. We can always revise it to another design later.
(Going with either LinearProgressIndicator, or a toast, for now will mean we spend basically zero effort on implementing the specifics of the UI — so there'll be essentially zero wasted work if we later go with another design.)
I'm about to file another issue for a toast with more detailed information, which might be too technical-looking for a general release but is fine for this beta and may help us pin down remaining issues in failing to connect.

A toast might be easier to apply in all the right places than the LinearProgressIndicator solution.
The toast solution requires fewer things of the surrounding UI, I think. To show a toast, you just need to know which account (if any) the foregrounded page belongs to and whether its data is stale. But to show a LinearProgressIndicator, you need to know that plus (if the page is per-account) you need the page to have an app bar. And for a nice experience, you might need to adapt to specific things about the app bar, like what color it is. Also I could imagine a future where some app bars use Material widgets and some don't.
I guess concretely, consider the lightbox, which is a per-account page: a saturated blue loading indicator might look jarring on the gray, semitransparent surface of the lightbox app bar, so we might want to tone that down. Also, the lightbox app bar isn't even always visible, because you can tap it to make it go away. I think something like a toast might be the only thing that really works on the lightbox page, actually. And if we're doing it on that page, we might as well do it everywhere; what do you think?

The toast solution requires fewer things of the surrounding UI, I think. To show a toast, you just need to know which account (if any) the foregrounded page belongs to and whether its data is stale. But to show a LinearProgressIndicator, you need to know that plus (if the page is per-account) you need the page to have an app bar. And for a nice experience, you might need to adapt to specific things about the app bar, like what color it is.

True, yeah.

"Hi gnprice,
Below is the proposed solution for issue #465. Please have a look at it and let me know so that I can move forward in integrating it with the Zulip beta application.
https://github.com/zulip/zulip-flutter/assets/112092492/49670e0e-53ae-47bb-94c2-f1d1d2214306

@satyam372 Hello, welcome. You asked the same question in a chat thread, and I've just replied there:
https://chat.zulip.org/#narrow/stream/48-mobile/topic/.23F465.20Show.20.22Connecting.E2.80.A6.22.20banner.20.28or.20equivalent.29.20when.20serv.2E.2E.2E/near/1762891


Consumer Group

I want to know is there any difference with Sarama and Sarama Cluster? I Sarama itself has a consumer group interface in its GoDoc so can we use it? I am asking this because in the same GoDoc Sarama refer to Sarama Cluster for group consuming.

Sarama cluster is going to be deprecated in favor of Sarama consumer groups -> https://github.com/bsm/sarama-cluster/issues/262#issuecomment-419878258
Please use sarama. I will clean up sarama go-doc soon.


chore: refactor yaml.ts file for readability and testability

Description
We want to leverage package.json to defined RBAC settings that will work with RBAC and create an excellent example test.
Describe the solution you'd like
Given we want to create a reproducible way to generate scoped RBAC
When we define the RBAC in package.json and run a build
Then we have our customized RBAC and RBAC rules are in values.yaml file
BUT, before moving on the adding the custom RBAC settings, we need to refactor yaml.ts for readability and testability.
...
End to End Test:
(See Pepr Excellent Examples)
Related Issue
Fixes  #1238

Relates to #814
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[X ] Other (security config, docs update, etc)

Checklist before merging

[ ] Unit, Journey, E2E Tests, docs, adr added or updated as needed
[ ] Contributor Guide Steps followed

Closed to focus on higher priorities


rofi -show

please change the script used to launch the plugin in the main page from :
rofi -modi file-browser -show file-browser
to use file-browser-extended not file-browser where file-browser is the built-in plugin of rofi,
thanks

Oh yeah, I must have forgotten that. Thanks for reminding me.


Preserve zoom level

I reduce the size of the elements by "Strg" + "-" to fit more content. But after a restart one gets the standard size again, but when I insert "Strg" + "-" then, it gets smaller as it was before the restart. So somewhere the zoomlevel is stored but not applied.
I guess it would be nice to furthermore access the zoomlevel in the app settings for users not familiar with the shortcuts.

Duplicate #1421


feat: add JSON logging

/closes #929
/claim #929

Hello @neo773, can you add this as a flag to the server flags?

@tolgaOzen
Are you talking about serve.go?
So a CLI flag to set override the config output?

It will be sufficient to add it here as log-output:
// pkg/cmd/flags/serve.go
// LOG
flags.String("log-level", conf.Log.Level, "real-time logs of authorization. Permify uses zerolog as a logger")
if err := viper.BindPFlag("logger.level", flags.Lookup("log-level")); err != nil {
	panic(err)
}
if err := viper.BindEnv("logger.level", "PERMIFY_LOG_LEVEL"); err != nil {
	panic(err)
}

@tolgaOzen
Pushed a new commit, can you check now?

Looks great. Thank you for your contribution.


Update README.md add Badges

Description of PR
I have tried improving the readme file by adding badges proving the various functionalities, Code and dependencies.
Relevant Issues
Fixes #2352
Checklist

[ ] Compiles and passes lint tests
[ ] Properly formatted
[ ] Tested on desktop
[ ] Tested on phone

Please read the Contributing Guidelines before creating a PR


Compare Crawler + VM to Local Data

I figured that it made more sense to separate this task from testing the crawler (#9) since they're fairly different when it comes to methodology (making sure the extension works vs. showing that the environment in which the extension is running is not disruptive). This will make it easier for us to track our progress on this issue instead of putting everything into #9.
As a reminder, this task is essentially to provide justification for our crawling methodology (crawler + VM to replicate regular user experience). The goal is to show that including these two variables doesn't significantly impact the data that we collect, i.e., that it's a fair representation of what a regular user with Privacy Pioneer would collect. While @atlasharry is finalizing the test list, I am making progress on this front. @atlasharry has also volunteered to collect data in Korea in addition to any results that I get.

I've finished the scripts for the comparisons that we'll be making when comparing local data vs. VM data vs. VM + Crawler data. The Google Colab can be found here in our Google Drive. Instructions for running the script are found at the link, so when the time comes, @atlasharry should be able to just upload his data, run the script, and get some results. I will also probably do my own data collection, but that will probably be after the ground truth analysis for #9 as discussed last Friday.
Here's a quick rundown of why this script was made, and what it's meant to do:

We needed a way to be able to compare a website's treatment of a user depending on where Privacy Pioneer was collecting information from. However, natural fluctuations of web requests posed a challenge; crawling each location once would oftentimes result in major discrepancies between data collected. The vast majority of these discrepancies could be explained by the requests simply not existing for a particular run, but that wouldn't be a very convincing argument for the validity of our crawl procedure. How would we know that websites weren't treating users on a VM completely differently?
The simplest way to resolve this would be to visit the website multiple times. But then, how do we know that the requests aren't differing massively between runs? To solve this, I created some data analysis scripts.

The general idea of my scripts was to do a couple of things. I've also attached a little graphic that helped me visualize my process.

Collect Privacy Pioneer data for each website from a test list multiple times. This data will be added to a "data pool", which will help us identify unique domains that are responsible for different categories of evidence.
Create a "profile" of every website that we're testing on. Each website URL has it's own dictionary, which then has sub-dictionaries for each category, which contains lists of unique domains that have been encountered. These profiles serve to answer the question, "what requests can I expect to see for a given category when visiting a site?"
To finish the profiles, we take note of how many times each URL appeared on a site for a category.
Once we have an idea of what a site looks like (i.e., we've collected 3 runs for twitter.com both locally and in the cloud), we can compare these profiles across treatment groups, while excluding outliers, which we've defined to be requests that only occured once.
I've also attached an example of what one of the resulting tables should look like. Proportions are also calculated, as well as statistics for the outliers.

Nice work, @dadak-dom!
The core measure we will be using are precision, recall, and F1 score.

As we discussed, we will proceed per @dadak-dom's two step analysis approach:
1. Privacy Pioneer + VM + Crawler "internal" analysis
In other words, does Privacy Pioneer work properly when adding a VM and the Crawler? This analysis is similar to just a straight Privacy Pioneer analysis except that we add VM + Crawler.
2. Privacy Pioneer + VM + Crawler vs Privacy Pioneer analysis
This is the second step, in which we check how much, if anything, of the analysis is incorrect if we add a VM and Crawler. To account for the natural fluctuation of site loads, @dadak-dom will run Privacy Pioneer + VM + Crawler and Privacy Pioneer each three times. If an analysis instance/result is showing up two (or three) times, we can count it. Otherwise, it might be natural fluctuation. We will also get a sense of the rate of fluctuation by comparing the intra-run differences (i.e., the rates inside the set of three runs of Privacy Pioneer + VM + Crawler and Privacy Pioneer). @atlasharry will repeat this test once he is in Seoul.
3. Misc
For both steps, we should calculate precision, recall, and F1. We should only calculated these scores from the positive instances and not the negative instances (i.e., not use weighted scores).
As @dadak-dom mentioned, this may take some time, which, however, is time well spent and will pay off later. @atlasharry will help @dadak-dom with the analysis.
We can also do some spotchecks after we have done the crawls.

I am closing this issue to continue all testing discussions in #9.


Fixed issues with enums

Short description outlining key changes/additions
Fixed problem with enums
JIRA Ticket Number
Type of change

[x] Bug fix
[ ] New feature
[ ] Breaking change
[ ] Infrastructure change

Pull request checklist

[ ] I have added unit tests for new code that I have added
[ ] I have added/updated functional tests where appropriate
[ ] The code follows our coding standards

An exhaustive list of peer review checks can be read here

CI: Security warnings found!


Replace cache pull to cache fetch

Why don't we use Git approach:
dvc cache fetch instead of dvc cache pull
dvc cache pull --> dvc cache fetch && dvc checkout

I realized that dvc cache pull is already working as dvc cache fetch && dvc checkout
So, we should think about introducing dvc cache fetch without checking out.

I totally agree. I'll introduce 'dvc cache fetch' soon, it is a quick addon after all.


Adding more OpenXR runtimes

Installed runtimes do not advertise themselves in any useful way (only the currently active one is easy to find).
What we do to find runtimes that may be installed, but aren't currently active, is to keep a list of paths to their manifest files.
We need to add search paths for existing runtimes that this program may not know about.

Sadly, most runtime are still not installing their path in the registry, despite the key to do so being described since the 1.0.18 OpenXR Loader spec.
On my system the only runtime that installed a value in HKEY_LOCAL_MACHINE\SOFTWARE\Khronos\OpenXR\1\AvailableRuntimes is Oculus.

I did not meant the available runtime key. I meant the registry from each program to grab it's install location dynamically. And yeh it's pretty sad they do not list in available runtimes, wglhich would make things a lot easier

Well that was supposed to be a temporary solution 😅

Latest SteamVR beta now also register itself in the available runtimes. It only does it when you accept the validated UAC prompt that will set SteamVR as default.

https://store.steampowered.com/news/app/250820/view/5641338588648488019


[AVM Module Issue]: ServiceCidr not working

Check for previous/existing GitHub issues

[X] I have checked for previous/existing GitHub issues

Issue Type?
Bug
Module Name
avm/res/container-service/managed-cluster
(Optional) Module Name if not listed above
No response
(Optional) Module Version
No response
Description
Parameters serviceCidr and dnsServiceIP at not properly working.
When I set serviceCidr to <IP_ADDRESS>/16 and dnsServiceIP  to <IP_ADDRESS>, I receive a validation error saying that the DNS service IP does not belong to the range <IP_ADDRESS>/16.
(Optional) Correlation Id
No response

Any news?

Hi @zensqlmonitor, I will check the bug as soon as possible.

Hi @zensqlmonitor
I suspect the vnet in which you try to deploy your cluster has the address prefix of <IP_ADDRESS>/16, this is set in some of the dependencies bicep files. If you are trying to configure a serviceCidr and dnsServiceIP (or even podCidr) with a range that does not fit within the address prefix of the vnet you use, the error you describe is exactly the on you get.
Cheers!

Hi @ruudje123, no, the vnet I'm using has not this address prefix.
My target is to set a service CIDR range in such a way that it does not overlap with the VNet IP range where the AKS cluster is hosted. I'm using this for EKS in AWS as a best practice.
For example, I'm using <IP_ADDRESS>/22 for the VPC and <IP_ADDRESS>/24 for the service IP address range.
Previously, Amazon EKS automatically chose a value for this range based on the primary CIDR block of the Amazon VPC used by the cluster. However, this approach sometimes caused conflicts with other IP ranges in use across the network, especially for customers with VPCs peered to on-premise networks or other Amazon VPCs.

Hi @zensqlmonitor
I assume you use azure-cni-overlay to plug your cluster into your vnet?
https://learn.microsoft.com/en-us/azure/aks/azure-cni-overlay?tabs=kubectl
Important settings for the template are networkPlugin, networkPluginMode, networkPolicy you can use 'azure' for plugin and policy and 'overlay' for plugin mode.
With Azure CNI Overlay if <IP_ADDRESS>/22 is your vnet you should be able to use <IP_ADDRESS>/24 as serviceCidr and also <IP_ADDRESS> as DNS IP. I would also configure a podCidr so none pods are exposed externally without an ingress loadbalancer.
If you are like me don't want the load balancers exposing the services or ingress not be in the same subnet as worker nodes, there is an annotation of the service manifest that allows you to configure this. I have yet to find the parameter I can configure in AKS to achieve this. the annotation is:
service.beta.kubernetes.io/azure-load-balancer-internal-subnet: "subnetname"
Hope it helps.

No, I'm using Azure CNI and that configuration works when using the portal but not with the AVF.

VNet: <IP_ADDRESS>/22
Service CIDR: <IP_ADDRESS>/24
DNS service IP: <IP_ADDRESS>

@JPEasier any thoughts ?

Hi, any news ?

Hello @JPEasier, any assistance found for this ?
Thanks.

Hi @zensqlmonitor I just tested this myself and did not get any issues. Azure CNI,
My networking looks like:
param vNetAddressPrefix = '<IP_ADDRESS>/16'
param aksPodSubnetPrefix = '<IP_ADDRESS>/20'
param aksLBSubnetPrefix = '<IP_ADDRESS>/28'
param serviceCidr = '<IP_ADDRESS>/24'
param dnsServiceIP = '<IP_ADDRESS>'

Can you test again maybe and let us know which version of the module you are using?

@PixelRobots @JPEasier
Looks fine with the version 0.1.3:

Awesome news. Should be ok to close this issue. Glad it's all working for you.
Also keep an eye out on the module as I have been adding some new features!