[Task]: Backend Test Cases for Appsmith Server functionalities for Native Saas Integrations

SubTasks

This is a parent task placeholder for all the test cases that will be added throughout this implementation for the Appsmith server. (More details and granular level tasks will be created once we do the implementation)

Adding a ballpark estimate for now.

Hey team! Please add your planning poker estimate with Zenhub @AmanAgarwal041 @sneha122


Realm fails to install via CocoaPods. Problem with build.sh file on El Capitan.

$ pod install
Updating local specs repositories
Analyzing dependencies
Downloading dependencies
Installing Realm (0.95.3)
[!] /bin/bash -c
set -e
sh build.sh cocoapods-setup

: command not found
: command not found
: invalid option namet: pipefail
: invalid option4: set: -
set: usage: set [--abefhkmnptuvxBCHP] [-o option] [arg ...]

The result doesn't depend on the version of Realm.
I've tried:
pod 'Realm'

pod 'Realm', '=0.95.2'

pod 'Realm', '=0.95.0'

source 'https://github.com/CocoaPods/Specs.git'
platform :ios, '9.0'
inhibit_all_warnings!
use_frameworks!
pod 'RealmSwift', '0.95.2'

pod --version
0.39.0

All other pods work well. I guess, it's somehow related to that I have El Capitan, OS X 10.11 (15A284).
I reinstalled a public release from AppStore again. It didn't change anything.
As I see build.sh file has Windows-style line endings. Therefore, I can't understand why everyone else has no problems with CocoaPods version on Realm.
Using static framework is not an option for me because of 3rd party libraries that depend on Realm CocoaPod.

are you using a custom shell, by any chance? Realm's installation script depends on bash.

$ echo $SHELL
/bin/bash

We've had one report of this happening several months ago as well, but we were never able to reproduce the issue or figure out why it was happening. See #1683.
Can you think of anything about your computer's configuration that might cause this @asqar? I still have no clue why this could happen.

I installed beta version of El Capitan. Then upgraded to the public release from the Mac AppStore.
Yesterday I re-installed El Capitan from the Mac AppStore again, bus it didn't help.
I've tried to reinstall CocoaPods several times.
My ruby version is:
$ ruby -v
ruby 2.2.2p95 (2015-04-13 revision 50295) [x86_64-darwin15]

$ which sh
/bin/sh

I have problems with Homebrew updates if it matters.
$ brew update
error: The following untracked working tree files would be overwritten by merge:
	Library/Homebrew/extend/enumerable.rb
Please move or remove them before you can merge.
Aborting
Error: Failure while executing: git pull --quiet origin refs/heads/master:refs/remotes/origin/master

I've seen similar errors in shell scripts in other contexts when the line endings of the files were incorrect (e.g., Windows-style CRLF line endings on Unix). Would you mind posting the output of the following command so we can see if the script has the expected line endings:
xxd scripts/swift-version.sh | head

As I worried and @bdash suggested, I had some misconfigured settings for line endings.
I ran:
git config --global core.autocrlf input

Then completely re-installed Homebrew and Ruby.
sudo rm -rf /usr/local/Cellar /usr/local/.git && brew cleanup
ruby -e "$(curl -fsSL https://raw.githubusercontent.com/Homebrew/install/master/install )"

brew install ruby

Now everything is cool.

Wow, I'm happy you two were able to resolve this. The previous report of this issue had really puzzled me.

Hey this was enough for me
git config --global core.autocrlf input
didnt have to re install ruby and homebrew

I suffered the same issue, to fix it I removed and re-installed cocoapods by gem instead of brew and it works.


Exclude id from model fields to avoid overriding the id: type

Description
Excluding the id from model_fields when MAP_AUTO_ID_AS_GLOBAL_ID is set to True to allow the previously set annotation to handle the id without overriding it.
Types of Changes

[ ] Core
[x] Bugfix
[ ] New feature
[ ] Enhancement/optimization
[ ] Documentation

Issues Fixed or Closed by This PR
#372
Checklist

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] I have tested the changes and verified that they work and don't break anything (as well as I can manage).

Codecov Report
Patch coverage: 60.00% and project coverage change: -0.05% :warning:

Comparison is base (d77771a) 88.00% compared to head (926df2e) 87.95%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #373      +/-   ##
==========================================
- Coverage   88.00%   87.95%   -0.05%
==========================================
  Files          33       33
  Lines        2943     2948       +5
==========================================
+ Hits         2590     2593       +3
- Misses        353      355       +2

Files Changed
Coverage Δ

strawberry_django/type.py
93.00% <60.00%> (-0.85%)
:arrow_down:

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Picture-in-Picture feature?

A picture-in-picture mode would be neat, like how one can do with videos in Firefox.

I also thought about this, but I don't think it's possible. It only works for <video> elements.
https://developer.mozilla.org/en-US/docs/Web/API/Picture-in-Picture_API

You actually can get this to work by creating a <video> element backed by a <canvas>. I did an experiment with Webamp (https://webamp.org) where i got this working: https://github.com/captbaritone/webamp/pull/791/commits/0faf3095c3b64882c6400a8ab2273e5bcddc24bb
But I never shipped it.
Full (unmerged) PR is here: https://github.com/captbaritone/webamp/pull/791
Something happened to this git history so the PR is confusing, but look at the last few commits.

@captbaritone Interesting, thanks! It seems the key is HTMLCanvasElement.captureStream(). I will try this. It seems like such a cool idea, why didn't you ship it?

To be honest, I can't recall. I think it was unclear how it would be surfaced in Webamp (since Winamp did not have pip) and some concern around browser compatibility/performance. I think the APIs of Butterchurn should allow this to be done without needing support for it in the library itself.

I have this working on my website and it works perfectly (on all browsers) by capturing the canvas stream into a video element, basically using the technique shared by @captbaritone above (thank you!), and there are other tutorials around dealing with  the same thing. The integration was in fact easy. I did it within a single React hook of less than 100 lines, including playback control and resize handlers in the pip window.
However I don't think that this feature should be the responsibility of the butterchurn library, rather the app developer incorporating the libary into their project.


Json serialization

Adds features based on topic #3

Coverage decreased (-0.2%) to 84.0% when pulling c7ad13043cd8283fd4e97fe7ade638b151995304 on JSON_serialization into 5c64ee978817dc637386500d86cc5cc635b7a8ce on master.


Cuckoo 1.2 vs Volatility 2.4

Hi all,
I'm trying Cuckoo 1.2 with Volatility 2.4; I'm running on Ubuntu 14.10 and the error I'm getting during memory dump processing is:
[modules.processing.memory] ERROR: Generic error executing volatility Traceback (most recent call last): File "/home/alex/cuckoo/modules/processing/memory.py", line 1047, in run results = vol.run() File "/home/alex/cuckoo/modules/processing/memory.py", line 960, in run if self.voptions.ssdt.enabled: AttributeError: Config instance has no attribute 'ssdt'
From what I can tell the Volatility 2.4 doesn't have the ssdt.py under volatility/plugins/malware/ as the memory.py module from Cuckoo implies.
NOTE: one can check this here hxxps://github.com/volatilityfoundation/volatility/tree/master/volatility/plugins/malware.
Volatility still has SSDT as a valid parameter, but this plugin (ssdt.py) is another location, volatility/plugins/ (in the top folder for plugins). Could this be the cause of the error above?
Thank you,
Alex.

I've had the same issue, and since I didn't need anything in Volatility
2.4, I used 2.3 with no problem.
On Sunday, March 8, 2015, The-Betrayer<EMAIL_ADDRESS>wrote:

Hi all,
I'm trying Cuckoo 1.2 with Volatility 2.4; I'm running on Ubuntu 14.10 and
the error I'm getting during memory dump processing is:
[modules.processing.memory] ERROR: Generic error executing volatility
Traceback (most recent call last): File
"/home/alex/cuckoo/modules/processing/memory.py", line 1047, in run results
= vol.run() File "/home/alex/cuckoo/modules/processing/memory.py", line
960, in run if self.voptions.ssdt.enabled: AttributeError: Config instance
has no attribute 'ssdt'
From what I can tell the Volatility 2.4 doesn't have the ssdt.py under
volatility/plugins/malware/ as the memory.py module from Cuckoo implies.
NOTE: one can check this here hxxps://
github.com/volatilityfoundation/volatility/tree/master/volatility/plugins/malware
.
Volatility still has SSDT as a valid parameter, but this plugin (ssdt.py)
is another location, volatility/plugins/ (in the top folder for plugins).
Could this be the cause of the error above?
Thank you,
Alex.
--
Reply to this email directly or view it on GitHub
https://github.com/cuckoobox/cuckoo/issues/495.

Thank you copeland, in truth one can do most of the stuff with Volatility 2.3 yet the change log for Cuckoo 1.2 says it should integrate with Volatility 2.4 - hence my question.

It does not look like an error with volatility, but more regarding your configuration file. Did you update your configuration files when updating to Cuckoo 1.2 ?
Could you check that your config file "memory.conf" contains an "ssdt" section like the following:
[ssdt]
enabled = yes
filter = on

Thanks a lot for the hint gaelmuller - this was it! We can close the thread.


Remove img requests spam for Logo, AlarmMsg and ServiceMsg on webserver

Description
Every second, several requests were sent to the web server to load the same images over and over again.
The round image next to alarm messages and service messages is reloaded every second. Even if the image does not change.
Furthermore, the logo is loaded twice every second. These requests are absolutely useless and can be prevented with a few small adjustments.
Related Issue
None
Types of changes

[ ] Docs change / refactoring / dependency upgrade
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Alternate Designs

Replace images by css
base64 embedded images

Possible Drawbacks
Nothing known
Verification Process
No function / parameter has been changed. Only the execution of the function. No dependencies available.
Release Notes

WebUi optimisation through removal of redundant img-requests

Contributing checklist

[x] My code follows the code style of this project.
[x] I have read the CONTRIBUTING and LICENSE document.
[x] I fully agree to distribute my changes under Apache 2.0 license.

First of all, thanks for your PR. However, the way you try to integrate it into pur development ecosystem is not correct. Please have a look how we generate/prepare WebUI Patches or provide your proposal as a diff/patch file so that we can replicate your proposed change. So, in the current form I cannot integrate this PR. So please revise!

@Steinweber I just looked at your PR and reworked it according to our WebUI patchset ecosystem. Please see if all necessary changes are integrated in the final patchset:
https://github.com/jens-maus/RaspberryMatic/blob/d978e0b36d173451a8ad10139d83fe8573d63101/buildroot-external/patches/occu/0153-WebUI-Fix-ImproveImgUpdates.patch#L1-L85
I wil merge this PR now but please reopen this PR or comment on it if you identify changes that you feel are missing. And also please have a look how your changes have been integrated by me now so that you can comply better with a potential new PR you want to submit in future.

I think I have understood how it works. However, some new questions have arisen in the meantime, which I still have to clarify.
Maybe you have the answers for me.

Where do the *.org files come from? Current status of eq-3/occu or the file that is delivered in the current RPMatic version?
i run create_patch.sh directly on the RPMatic. However, it has problems with the --label parameter. If I remove it, it works.

Basically:
Do you have any tips on the easiest way to develop (if you have to work on Windows)?
I currently do it like this:

Load the Git repo into my IDE
RPMatic installed in VM
Edit files from VM
Upload repo in subfolder to create patch

x I haven't found a way to test the patch in a new VM yet.
very cumbersome - but works somehow

@Steinweber This is not a discussion fora here, but well, I will answer your questions:

Where do the *.org files come from? Current status of eq-3/occu or the file that is delivered in the current RPMatic version?

It depends where to take the *.orig files from. Optimally you would take them from eq-3/occu or jens-maus/occu directly. However, if the new patch will patch something in the very same line (or surrounding) then you would need to take the latest file from a previous webui patch as the new *.orig file.

i run create_patch.sh directly on the RPMatic. However, it has problems with the --label parameter. If I remove it, it works.

Basically: Do you have any tips on the easiest way to develop (if you have to work on Windows)?

Install a virtual Linux machine to perform development there and don't use Windows directly. That's the deal!

I currently do it like this:

Load the Git repo into my IDE
RPMatic installed in VM
Edit files from VM
Upload repo in subfolder to create patch

The procedure should be as follows:

Create a Linux VM for development (e.g. Ubuntu 20.04)
Use ONLY this Linux VM for development
Checkout the RaspberryMatic github repo in this Linux-vm
Compile your RaspberryMatic build via make raspmatic-XXX-release where XXX corresponds to your target platform
Install the update and check the changes.

x I haven't found a way to test the patch in a new VM yet.

You can only do this by re-creating the build. Thus, something like

rm -rf build-raspmatic-XXX/build/occu*
make raspmatic-XXX-release

I hope to create some more developer documentation on this matter soon, btw.


feat: port GroupTheory.GroupAction.ConjAct

This compiles and lints now, BUT...

There were some very slow simps here, that I had to massage with simp only and more. I didn't expect these to be so slow.
I had to add a shortcut instance for MulAction (ConjAct G) G in order for TC inference to find it in the declaration stabilizer_eq_centralizer.
There were several simp lemmas that simpNF said it could prove for which I just removed the simp attribute.

I think we may at least want to consider addressing the first two problems before merging.

bors merge


Habitat installation

Habitat-lab version
Hello
I followed all instructions to install habitat, I did not face any problem in installation.
I tried to run the following task
https://github.com/facebookresearch/habitat-lab/blob/main/examples/interactive_play.py
but I found this problem related to the kernel "The kernel 'habitat (Python 3.9.18)' died. Click here for more info. View Jupyter log for further details"
Please tell me how I can solve this problem ?
Thanks in advance
❓ Questions and Help

This looks like a habitat-lab issue. Please re-open there if you are still having issues. We'll also need more information (e.g. log output and system info) to start helping.


loader: Fix filename and native lib dir sequence for log

Hi @rpavlik , could you help to review this PR? Thanks.

Looks good, can you add a changelog fragment in changes/sdk? See readme in there

Looks good, can you add a changelog fragment in changes/sdk? See readme in there

Hi @rpavlik , I added a new commit to add changelog fragment based on previous PRs. If it doesn't meet requirement, please let me know.


Upgraded springboot version & completed migration of existing features

This PR completes the migration of the existing features. (Related to issue #19 )

Hi @xcapdevila,
thanks so much for this PR!! This really helps me to focus on the core development of messenger4j!
The next 3 weeks I'm very very busy so I will not have the time to review it but afterwards it's the first thing I will work on. It's really valuable to finally merge the rewritten quick-start-template to master to make it available to everyone.

Thank you so much @xcapdevila for your great effort to migrate this project! Much appreciated!
Thanks @Plotron for reviewing!


HUD example compiles but crashes

This crashes as soon as I launch the example with the following traceback:
fatal error: unexpected signal during runtime execution
[signal 0xb code=0x1 addr=0x0 pc=0x7f90bf226ff0]

runtime stack:
runtime.throw(0x7bac40, 0x2a)
	/home/faide/tools/go/src/runtime/panic.go:527 +0x90
runtime.sigpanic()
	/home/faide/tools/go/src/runtime/sigpanic_unix.go:12 +0x5a

goroutine 1 [syscall, locked to thread]:
runtime.cgocall(0x60d8e0, 0xc8200c7ad0, 0x0)
	/home/faide/tools/go/src/runtime/cgocall.go:120 +0x11b fp=0xc8200c7a88 sp=0xc8200c7a58
github.com/go-gl/gl/v2.1/gl._Cfunc_glowDrawElements(0x7f90bef73760, 0x600000004, 0x1403, 0x0)
	github.com/go-gl/gl/v2.1/gl/_obj/_cgo_gotypes.go:12227 +0x35 fp=0xc8200c7ad0 sp=0xc8200c7a88
github.com/go-gl/gl/v2.1/gl.DrawElements(0x600000004, 0xc800001403, 0x0)
	/home/faide/go/src/github.com/go-gl/gl/v2.1/gl/package.go:19385 +0x45 fp=0xc8200c7af8 sp=0xc8200c7ad0
github.com/paked/webgl.(*Context).DrawElements(0xc8200f6000, 0x4, 0x6, 0x1403, 0x0)
	/home/faide/go/src/github.com/paked/webgl/webgl_gl2.go:775 +0x42 fp=0xc8200c7b18 sp=0xc8200c7af8
github.com/paked/engi.(*HUDShader).flush(0xc8200be000)
	/home/faide/go/src/github.com/paked/engi/shaders.go:289 +0x168 fp=0xc8200c7b58 sp=0xc8200c7b18
github.com/paked/engi.(*HUDShader).Post(0xc8200be000)
	/home/faide/go/src/github.com/paked/engi/shaders.go:294 +0x21 fp=0xc8200c7b68 sp=0xc8200c7b58
github.com/paked/engi.(*RenderSystem).Post(0xc82000e420)
	/home/faide/go/src/github.com/paked/engi/render.go:291 +0xce fp=0xc8200c7ca0 sp=0xc8200c7b68
github.com/paked/engi/ecs.(*World).Update(0xc820014720, 0x0)
	/home/faide/go/src/github.com/paked/engi/ecs/world.go:121 +0x2d5 fp=0xc8200c7d90 sp=0xc8200c7ca0
github.com/paked/engi.RunIteration()
	/home/faide/go/src/github.com/paked/engi/engi_glfw.go:194 +0x53 fp=0xc8200c7da8 sp=0xc8200c7d90
github.com/paked/engi.runLoop(0x7f90bc69f360, 0xc6fb60, 0x400)
	/home/faide/go/src/github.com/paked/engi/engi_glfw.go:226 +0xd9 fp=0xc8200c7e70 sp=0xc8200c7da8
github.com/paked/engi.Open(0x0, 0x71d4b0, 0x8, 0x0, 0x400, 0x280, 0x0, 0x0, 0x7f90bc69f360, 0xc6fb60)
	/home/faide/go/src/github.com/paked/engi/engi.go:68 +0xf4 fp=0xc8200c7ea0 sp=0xc8200c7e70
main.main()
	/home/faide/go/src/github.com/paked/engi/demos/hud/hud.go:108 +0xab fp=0xc8200c7f50 sp=0xc8200c7ea0
runtime.main()
	/home/faide/tools/go/src/runtime/proc.go:111 +0x2b0 fp=0xc8200c7fa0 sp=0xc8200c7f50
runtime.goexit()
	/home/faide/tools/go/src/runtime/asm_amd64.s:1721 +0x1 fp=0xc8200c7fa8 sp=0xc8200c7fa0

goroutine 17 [syscall, locked to thread]:
runtime.goexit()
	/home/faide/tools/go/src/runtime/asm_amd64.s:1721 +0x1

if I suppress the call to w.AddEntity(hudBg) it works so I suspect something in generateHUDBackground but I can't see what really differs from generateBackground which works fine.

What version of Go are you using?

 Hopefully they will have a fix out soon.
In the meantime, I'll try and debug this and get back to you when I have something new!

Alright. So it looks like e8d15d8b2af5087e65eaa47ed7a645994477254b broke the HUD. @EtienneBruines Do you have time to take point on fixing this (I haven't touched the shaders in forever.)?

@faide Thank you for reporting this issue. I will have time for this tonight (in about 6 hrs). Should be a relatively easy fix, though.
W.r.t. the Go 1.6, there has been made some progress (not by me) here.

OK, this is weird. Using the code from master, and go1.5.2, I'm seeing no runtime error whatsoever. (using go run hud.go)
May I ask how you're attempting to run it, @faide?
Are you able to replicate this bug, @paked?
@faide I see the callback goes all the way back to shaders.go:289, which is a specific shader defined for the HUD. This would explain why there's a difference between the HUD and the background. So this would have nothing to do with the example hud.go. I'm not quite sure yet why this would result in a GPU/OpenGL crash... (which is what's happening)

@EtienneBruines I was able to replicate this on OS X Yosemite, go1.5.3.
I'll try on Arch Linux now.

Then it must be a glitch on my part, causing it to "not crash". Investigating...

I am on Ubuntu
$ lsb_release -a
No LSB modules are available.
Distributor ID:	Ubuntu
Description:	Ubuntu 14.04.4 LTS
Release:	14.04
Codename:	trusty

here is a transcript of what I do once inside engi source code
$ git pull
Already up-to-date.
$ cd demos/hud/
$ go build -a
$ ./hud
... same crash as reported ...

I tried also with go 1.5.2 and got the same result. If you need any more system info (libs, kernel, whatever...) just ask
I verified:
github.com/paked/webgl (up to date)
github.com/go-gl/gl (up to date)

Thank you @faide.
Could you run glxheads in the terminal? This outputs some variables.

@faide Could you try this branch? https://github.com/EtienneBruines/engi/tree/hotfix_144
Probably something like:
git remote add EtienneBruines https://github.com/EtienneBruines/engi.git
git fetch EtienneBruines
git checkout EtienneBruines/hotfix_144

First part of the answer
$ glxheads :0
Name: :0
  Display:     0x2256010
  Window:      0x1600002
  Context:     0x2264c10
  GL_VERSION:  3.0 Mesa 10.1.3
  GL_VENDOR:   Intel Open Source Technology Center
  GL_RENDERER: Mesa DRI Intel(R) Ivybridge Mobile

I updated to your hotfix
and still crashes (github.com/paked/engi/shaders.go:255 being the last engi code called)
fatal error: unexpected signal during runtime execution
[signal 0xb code=0x1 addr=0x0 pc=0x7ff991c47ff0]

runtime stack:
runtime.throw(0x7ba060, 0x2a)
	/home/faide/tools/go/src/runtime/panic.go:527 +0x90
runtime.sigpanic()
	/home/faide/tools/go/src/runtime/sigpanic_unix.go:12 +0x5a

goroutine 1 [syscall, locked to thread]:
runtime.cgocall(0x60cfd0, 0xc820745ae0, 0x0)
	/home/faide/tools/go/src/runtime/cgocall.go:120 +0x11b fp=0xc820745a98 sp=0xc820745a68
github.com/go-gl/gl/v2.1/gl._Cfunc_glowDrawElements(0x7ff991994760, 0x600000004, 0x7ff900001403, 0x0)
	github.com/go-gl/gl/v2.1/gl/_obj/_cgo_gotypes.go:12233 +0x35 fp=0xc820745ae0 sp=0xc820745a98
github.com/go-gl/gl/v2.1/gl.DrawElements(0x600000004, 0xbf80000000001403, 0x0)
	/home/faide/go/src/github.com/go-gl/gl/v2.1/gl/package.go:19385 +0x45 fp=0xc820745b08 sp=0xc820745ae0
github.com/paked/webgl.(*Context).DrawElements(0xc8200f8000, 0x4, 0x6, 0x1403, 0x0)
	/home/faide/go/src/github.com/paked/webgl/webgl_gl2.go:775 +0x42 fp=0xc820745b28 sp=0xc820745b08
github.com/paked/engi.(*HUDShader).Draw(0xc820064150, 0xc8205c6f10, 0xc8205c6f50, 0xbf800000bf800000, 0xc800000000)
	/home/faide/go/src/github.com/paked/engi/shaders.go:255 +0x215 fp=0xc820745b68 sp=0xc820745b28
github.com/paked/engi.(*RenderSystem).Post(0xc82000e420)
	/home/faide/go/src/github.com/paked/engi/render.go:286 +0x415 fp=0xc820745ca0 sp=0xc820745b68
github.com/paked/engi/ecs.(*World).Update(0xc820014840, 0x0)
	/home/faide/go/src/github.com/paked/engi/ecs/world.go:121 +0x2d5 fp=0xc820745d90 sp=0xc820745ca0
github.com/paked/engi.RunIteration()
	/home/faide/go/src/github.com/paked/engi/engi_glfw.go:194 +0x53 fp=0xc820745da8 sp=0xc820745d90
github.com/paked/engi.runLoop(0x7ff98f0c0360, 0xc6eb60, 0x400)
	/home/faide/go/src/github.com/paked/engi/engi_glfw.go:226 +0xd9 fp=0xc820745e70 sp=0xc820745da8
github.com/paked/engi.Open(0x0, 0x71c9c0, 0x8, 0x0, 0x400, 0x280, 0x0, 0x0, 0x7ff98f0c0360, 0xc6eb60)
	/home/faide/go/src/github.com/paked/engi/engi.go:68 +0xf4 fp=0xc820745ea0 sp=0xc820745e70
main.main()
	/home/faide/go/src/github.com/paked/engi/demos/hud/hud.go:108 +0xab fp=0xc820745f50 sp=0xc820745ea0
runtime.main()
	/home/faide/tools/go/src/runtime/proc.go:111 +0x2b0 fp=0xc820745fa0 sp=0xc820745f50
runtime.goexit()
	/home/faide/tools/go/src/runtime/asm_amd64.s:1721 +0x1 fp=0xc820745fa8 sp=0xc820745fa0

goroutine 17 [syscall, locked to thread]:
runtime.goexit()
	/home/faide/tools/go/src/runtime/asm_amd64.s:1721 +0x1

I'm attempting to see if the HUDShader is to blame or not. Could you try again? (new commit in that branch)
(note that the goal of this test is not to see if it works, but to see whether or not it will crash)

@EtienneBruines That does not crash for me now.

OK, good to know. Then it's definitely the HUDShader.

Does the new commit fix the issue? (it works as intended on my pc, and if it doesn't crash, then we're all happy :-).

@EtienneBruines Works for me!

@faide Thank you for reporting this issue.
If it turns out that #145 didn't fix your issue, feel free to reopen. Happy to help.

I confirm it does not crash anymore


mount: /run/k3os/iso: wrong fs type, bad option, bad superblock on .... missing codepage ....

Describe the bug
When executing an ISO install of Harvester v0.2.0 on a HP ProLiant G9 server over iLO, I get the error seen on the below screenshot and in the title of this issue.

To Reproduce
Steps to reproduce the behavior:

Login to the iLO webinterface of the HP server
Attach the ISO, a locally downloaded ISO, to the server
When the server is booting up select to boot from the virtual cd-rom
go through the install ....

then already when Harvester is preparing the installation wizard steps, right before the initial install wizard page - where one states the server-name and so forth, errors referring to bad superblock or fs type can be seen.
Then if one continue, the error is seen again and blocks the installation from continuing after the entire installation wizard has been run through. And ===Installing Harvester==== is running.
Expected behavior
It can be fixed/worked around by entering the grub menu by hitting e and then adding rescue to the menu. When arriving at the rescue shell one can delete any potential partitions already present on the disk restart the installation and it works.
Environment:

Harvester ISO version: The official v0.2.0 release
Barae-metal installation
HP ProLiant G9 server

Additional context
It's my impression that this could be caused by Harvester not being able to successfully wiping any previous partitions on the disk one wants to install Harvester on.

The error message looks similar to https://github.com/rancher/k3os/issues/488 which happens to be a K3OS issue from you. Are they possibly related?
Inferring from the code, this is probably in https://github.com/harvester/harvester-installer/blob/deb8bd1eec15dfcd22163e59063b5d33b33d8ec9/k3os/install.sh#L254-L264 where it failed to get the ISO_DEVICE then failed the follow-up mounts.

Is there any progress?

@lin1005q Did you hit the same issue? Can you describe your setup/steps?

download iso (https://releases.rancher.com/harvester/v0.2.0/harvester-amd64.iso)
Copy the file to the USB flash drive.
The USB has installed ventoy
Turn on the computer
same error

@lin1005q
The way of burning the USB stick matters. Users had reported that some of the ways like the etcher tool not working.
Does it work if you prepare the USB drive with

dd if you use Linux/macOS
rufus in DD mode if you use Windows

For more background infos: https://github.com/pbatard/rufus/wiki/FAQ#why-doesnt-rufus-recommend-dd-mode-over-iso-mode-for-isohybrid-images-surely-dd-is-better
(We probably need to add the information to the FAQ docs and investigate how we can improve the compatibility in further releases if that's the case.)

close due to inactivity, also we have replaced the k3os with the newOS in our installer, please verify it using https://releases.rancher.com/harvester/master/harvester-amd64.iso and feel free to open a new issue if any similar issue was encountered.


Create README.es.md

This is a README file translated into Spanish (Spain).
The file translations/README.es.md has been created.

@microsoft-github-policy-service agree [company="{your company}"]

@microsoft-github-policy-service agree

This is a README file translated into Spanish (Spain).
The file translations/README.es.md has been created.

Looks Good To Me! 👆
Can You tell me Have you received this kind of error?

Can You tell me Have you received this kind of error?

I haven't received that error

Can You tell me Have you received this kind of error?

I haven't received that error

Oh okay Thanks for replying!c :)

@abrilurena when you have some time, could you review this translations PR? Thank you!

Hola, @sandra-laguna! espero que estes bien. Si tienes alguna duda, por favor, no dudes en preguntarme. Si te parecieron bien los cambios que sugeri, solo tienes que hacer click en "commit suggestion" para que se actualice el archivo original :D saludos!

¡Hola, @sandra-laguna! espero que estes bien. Ha pasado un tiempo desde la ultima vez que hablamos, me preguntaba si hay algo en lo que aun necesites ayuda? ✨


Provide cast support between int types and double

This add support for casting between int types and doubles

Double to int is defined as truncate towards zero and therefore possible without rounding
Int to double requires some rounding. Here i use the trick to cast the BigInt into float and back to see whether is is representable as float.

Also adds explicitly checks for FDouble type in some locations and removes it the comments on type checks in others, where the checks should be rather be handled inside the FloatDomTuple once we support multiple different fkinds.

Regarding the unittests, you are definitely right. But i would actually wait a bit longer with that, because once/if we extend to other floating type precisions, we might also have to rework/extend the casting between different precisions.


Bot Will Not Start

When I do the npm start command I am getting an error and it appears to me to be an issue with the token? But I just generated it. Am I missing a step that was not in the readme?
Here is the error I am seeing when I try and start.
npm start

<EMAIL_ADDRESS>start
node .

node:internal/modules/cjs/loader:1289
throw err;
^
SyntaxError: /home/mobbyg/gitstuff/streamer-alerts-discord-bot/config.json: Unexpected token / in JSON at position 90
at parse ()
at Module._extensions..json (node:internal/modules/cjs/loader:1286:39)
at Module.load (node:internal/modules/cjs/loader:1081:32)
at Module._load (node:internal/modules/cjs/loader:922:12)
at Module.require (node:internal/modules/cjs/loader:1105:19)
at require (node:internal/modules/cjs/helpers:103:18)
at Object. (/home/mobbyg/gitstuff/streamer-alerts-discord-bot/src/index.js:5:16)
at Module._compile (node:internal/modules/cjs/loader:1218:14)
at Module._extensions..js (node:internal/modules/cjs/loader:1272:10)
at Module.load (node:internal/modules/cjs/loader:1081:32)
Node.js v18.13.0

When I do the npm start command I am getting an error and it appears to me to be an issue with the token? But I just generated it. Am I missing a step that was not in the readme?
Here is the error I am seeing when I try and start.
npm start

<EMAIL_ADDRESS>start
node .

node:internal/modules/cjs/loader:1289 throw err; ^
SyntaxError: /home/mobbyg/gitstuff/streamer-alerts-discord-bot/config.json: Unexpected token / in JSON at position 90 at parse () at Module._extensions..json (node:internal/modules/cjs/loader:1286:39) at Module.load (node:internal/modules/cjs/loader:1081:32) at Module._load (node:internal/modules/cjs/loader:922:12) at Module.require (node:internal/modules/cjs/loader:1105:19) at require (node:internal/modules/cjs/helpers:103:18) at Object. (/home/mobbyg/gitstuff/streamer-alerts-discord-bot/src/index.js:5:16) at Module._compile (node:internal/modules/cjs/loader:1218:14) at Module._extensions..js (node:internal/modules/cjs/loader:1272:10) at Module.load (node:internal/modules/cjs/loader:1081:32)
Node.js v18.13.0

Hey @mobbyg sorry just seeing issue, have you solved this? Seems its not finding a bot token.
Steps to solve:
1 - Rename config.json.example to config.json
2 - Make sure you put your discord bot token in the config (KEEP THIS PRIVATE AND SAFE!!)
{
  "token": "xxxxxxxxxxxxxxxxxxxxxx",                                    // bot token
  "color": "#61CB2B",                                                           // embed color
  "footerText": "Developed By: Bankkroll",                          // embed footer text
  "footerIcon": "https://i.ibb.co/Bqs3jh7/6302741.png"     // embed footer icon
}


Any plan to support pot format in merger?

Dear @vorlif
Thanks for your great work. :)
Seems merger supports JSON but not pot format, any plan to support pot?

Thanks for the feedback @iredmail! Currently, there are no plans for something like this because there are just too many tools out there that already do the job perfectly.
If you use PoEdit for example, you can update your .po file via "Translation" -> "Update from POT file". Alternatively, there is the classic standard tool msgmerge for Windows, Linux and macOS, if you don't know this one yet?
With this, you can easily update your po files.
msgmerge -U your_old_translation.po latest_strings.pot

If you already knew these ways, what advantage do you see in performing it directly through xpreak.  :blush:

Thanks for sharing. :)


what is this line?

This is not an issue, it's only happening because the backend used doesn't have the Language & Currency dropdown or the top right links CMS components yet. This will be fixed with a backend update.

fixed by #1395


Nodes lose connectivity after being teared down for > 15 min

Stop node a)
Wait >15 min (15 min is the TTL for the lease in the kvstore)
Restart node a)
Verify the node can't connect to kube-dns, running on a different node, because coredns's node lost the route to reach node a).

Possible solutions:

Force key-value creation in etcd with cilium-operator for node a) based in k8s events. (I'm assuming we only do the creation of the key-value upon 'Create' events of that node but not upon 'Update' events)

which version does this affect?

which version does this affect?

@christianhuening all


Add volumes option for commands

Looks fine. Merged.

Could we also get documentation about this feature, please?

I've added some docs (http://vagga.readthedocs.org/en/latest/commands.html#opt-volumes). But overall I must be more rigorous and require documentation for these features.

Thanks


add shellcheck-disable for HOMEBREW_CORE_REPOSITORY

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[ ] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[x] Have you successfully run brew tests with your changes locally?

set shellcheck disable to avoid the warning message about HOMEBREW_CORE_REPOSITORY of brew style.
In /usr/local/Homebrew/Library/Homebrew/brew.sh line 341:
HOMEBREW_CORE_REPOSITORY="${HOMEBREW_LIBRARY}/Taps/homebrew/homebrew-core"
^----------------------^ SC2034: HOMEBREW_CORE_REPOSITORY appears unused. Verify use (or export if used externally).

set shellcheck disable to avoid the warning message about HOMEBREW_CORE_REPOSITORY of brew style.

@hyuraku Why is this not failing CI as-is 🤔? Would be good to fix that.

set shellcheck disable to avoid the warning message about HOMEBREW_CORE_REPOSITORY of brew style.

@hyuraku Why is this not failing CI as-is 🤔? Would be good to fix that.

It seems that the spellcheck did not execute.

It seems that the spellcheck did not execute on CI.

@hyuraku weird. https://github.com/Homebrew/brew/blob/7d566156311a03bd4b4434b8adcf822b378ef29c/Library/Homebrew/style.rb#L62-L66 and https://github.com/Homebrew/brew/blob/7d566156311a03bd4b4434b8adcf822b378ef29c/Library/Homebrew/style.rb#L166 would be useful places to look and add debugging to figure this out?

Thanks again @hyuraku!


BIGTOP-3857: Support parent directory default configuration for Hadoop rpm

BIGTOP-3857: Support parent directory default configuration for Hadoop rpm

Description of PR
How was this patch tested?
For code changes:

[ ] Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'BIGTOP-3638. Your PR title ...')?
[ ] Make sure that newly added files do not have any licensing issues. When in doubt refer to https://www.apache.org/licenses/

+1, tested on Ambari.

bigtop-packages/src/common/hadoop/hadoop.default should be removed if it is not used anymore? @timyuer.

bigtop-packages/src/common/hadoop/hadoop.default should be removed if it is not used anymore? @timyuer.

This is a missing point, and I'll fix it soon

bigtop-packages/src/common/hadoop/hadoop.default has been removed, and tested in Centos7(x86_64) and Ubuntu20.04(x86_64).  @iwasakims


Separate rebar_config loader interface from implementation.

Allow user to setup a custom config loader, with rebar env 'config_loader'.
Config loader must implement rebar_config behaviour

@jeanparpaillon ok if we close this since a different direction is being taken in #676?

Ok.


Added ARM template for Storage.

what is this file used for?

It's used for private CIs to generate Storage related credentials.

It's used for private CIs to generate Storage related credentials.

Is it able to work standalone? If not, I don't want to merge such a PR.


cmake: Add -gdwarf-4 at link time for gcc

Add a linker option for gcc to force dwarf symbols to be version 4.
Fixes #50106
Related to #35707
Signed-off-by: Jeremy Bettis<EMAIL_ADDRESS>

Please do not merge PRs without the assignee's approval (especially those marked with "Maintainer" label).

Please do not merge PRs without the assignee's approval (especially those marked with "Maintainer" label).

Noted, will look for this.


grabbing centermost article for each cluster

Pull request for partial title information.

Merged in different PR #7


Openweave Support

Support https://github.com/openweave/openweave-core with Zephyr:
openWeave is a home area network application protocol stack designed to enable asynchronous, symmetric, device-to-device, device-to-mobile and device-to-cloud communications for control path and data path messaging.

Closing due to age, I'm assuming.


[MacOS] Unable to display source file when debugging .net dll compiled on Windows with portable pdb format and published as nuget

Issue Description
I have a application .net 6 application compiled on Windows with portable pdb format and published as nuget in an internal company nuget server.
When executing the application in the VSCode debugger on a Mac I am able to put breakpoints and the execution stops when reaching the breakpoints BUT the debugger is unable to display the source file.
Instead it displays a message such as the following one:
Could not load source 'd:\b\0\193178\Sharpmake\Sharpmake\DebugProjectGenerator.cs': Incorrect format of 'source' message..
If I compile the application locally on the mac, the breakpoints works as expected
Steps to Reproduce

Compile an application with portable pdb format on Windows, publish it in a nuget package.

Install the nuget package on the mac

Get the source code for the app on the mac

Setup a launch.json for debugging the app on the mac

Put some breakpoint in the app source code

Run the the app the vscode debugger.

When reaching a breakpoint, it should display a message such as:
Could not load source 'd:\b\0\193178\Sharpmake\Sharpmake\DebugProjectGenerator.cs': Incorrect format of 'source' message..

Expected Behavior
Debugger should transform the paths so that a corresponding file can be found on the mac in the current workspace. Maybe transform backslashes in slashes and ignore all paths or part of it.
Actual Behavior
Logs

C# log
Post the output from Output-->C# here
C# LSP Trace Logs
Post the output from Output-->C# LSP Trace Logs here.  Requires `dotnet.server.trace` to be set to `Trace`
Environment information
VSCode version: 1.90.0
C# Extension: 2.31.19
Using OmniSharp: false
Dotnet Information
.NET SDK:
 Version:           8.0.301
 Commit:            1a0e9c0300
 Workload version:  8.0.300-manifests.f6879a9a
 MSBuild version:   17.10.4+10fbfbf2e
Runtime Environment:
OS Name:     Mac OS X
OS Version:  14.5
OS Platform: Darwin
RID:         osx-arm64
Base Path:   /usr/local/share/dotnet/sdk/8.0.301/
.NET workloads installed:
There are no installed workloads to display.
Host:
Version:      8.0.6
Architecture: arm64
Commit:       3b8b000a0e
.NET SDKs installed:
6.0.422 [/usr/local/share/dotnet/sdk]
6.0.423 [/usr/local/share/dotnet/sdk]
7.0.410 [/usr/local/share/dotnet/sdk]
8.0.301 [/usr/local/share/dotnet/sdk]
.NET runtimes installed:
Microsoft.AspNetCore.App 6.0.30 [/usr/local/share/dotnet/shared/Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 6.0.31 [/usr/local/share/dotnet/shared/Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 7.0.20 [/usr/local/share/dotnet/shared/Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 8.0.6 [/usr/local/share/dotnet/shared/Microsoft.AspNetCore.App]
Microsoft.NETCore.App 6.0.30 [/usr/local/share/dotnet/shared/Microsoft.NETCore.App]
Microsoft.NETCore.App 6.0.31 [/usr/local/share/dotnet/shared/Microsoft.NETCore.App]
Microsoft.NETCore.App 7.0.20 [/usr/local/share/dotnet/shared/Microsoft.NETCore.App]
Microsoft.NETCore.App 8.0.6 [/usr/local/share/dotnet/shared/Microsoft.NETCore.App]
Other architectures found:
x64   [/usr/local/share/dotnet/x64]
registered at [/etc/dotnet/install_location_x64]
Environment variables:
Not set
global.json file:
Not found
Learn more:
https://aka.ms/dotnet/info
Download .NET:
https://aka.ms/dotnet/download

Visual Studio Code Extensions

Extension
Author
Version
Folder Name

csdevkit
ms-dotnettools
1.6.8
ms-dotnettools.csdevkit-1.6.8-darwin-arm64

csharp
ms-dotnettools
2.31.19
ms-dotnettools.csharp-2.31.19-darwin-arm64

remote-explorer
ms-vscode
0.4.3
ms-vscode.remote-explorer-0.4.3

remote-ssh
ms-vscode-remote
0.110.1
ms-vscode-remote.remote-ssh-0.110.1

remote-ssh-edit
ms-vscode-remote
0.86.0
ms-vscode-remote.remote-ssh-edit-0.86.0

vscode-dotnet-runtime
ms-dotnettools
2.0.5
ms-dotnettools.vscode-dotnet-runtime-2.0.5

vscode-xml
redhat
0.27.1
redhat.vscode-xml-0.27.1-darwin-arm64

@jspelletier Can you add a sourceFileMap section to your launch.json?
{
   "program": ....,
   "sourceFileMap": {
      "d:\\b\\0\\193178\\Sharpmake\\Sharpmake\\": "<Path-to-Sharpmake-on-macOS>"
   }
}

This will help the debugger map from the Windows path to the macOS path.
Let me know if this fixes your issue.

Hi,
The following works perfectly:
"sourceFileMap": {
"d:\b\0\193178\": "/Users/jspelletier/gitlab/sharpmakeextended"
}
Thanks a lot!

@WardenGnaw Even if I closed it, I would like to know if we can specify wildcards or something like that in the left part of the path.


[MPG] Sample generation for Typespec

Now mgmt SDK from swagger has auto-generated sample (e.g. here), and we hope we could also have similar feature for Typespec.

The best solution is TCGC could provide source examples like m4 does

Move to https://github.com/Azure/autorest.python/issues/2622


What does this mean?

from section 2.1

If two local variables x and y are explicitly uninitialized, then on every path x’s definite first use must precede y’s definite first use.

What does this mean? y isn't mentioned at all prior to this or after this and I assume both x and y are placeholders. As such when it says x must precede y it's not clear to me which is which. e.g. in the subsequent example with uninitialized variables j and s how can I tell which one is the x that must be initialized first.
My best guess is that is intended to be interpreted as "If two local variables x and y are explicitly uninitialized and one one code path x's definite first use precedes y's definite first use then x's definite first use must precede y's definite first use on all code paths". However I can't see a justification provided for that assertion and can't intuit a good reason for it myself.

Good point, thanks. The point is that you have to initialize variables in declaration order.
I'll update the three bullets as follows:

A declaration of a local variable x that is initialized with = uninitialized; allocates stack memory as usual but does not invoke a constructor.
Every path containing x’s declaration must contain a definite first use of x before the declaration or initialization of another local variable, and that definite first use must be either to pass x to another function’s out parameter, or else be an assignment-expression to x (which can be from a single-object ex-pression or a possibly-empty initializer_list) that performs a single constructor call (usually direct-initialization) of x with the right-hand side expression treated as the argument to x’s constructor.
If two local variables x and y are declared in that order and are explicitly uninitialized, then on every path x’s definite first use must precede y’s definite first use.

And I'll add this Note:

The requirement to initialize in declaration order preserves that variable lifetimes continue to nest correctly, so that a later-declared local variable y can safely refer to a previously-declared variable x that will outlive y by construction.

I'll be in the next version of the PDF I push. Thanks!

Good point, thanks. The point is that you have to initialize variables in declaration order.
I'll update the three bullets as follows:

A declaration of a local variable x that is initialized with = uninitialized; allocates stack memory as usual but does not invoke a constructor.
Every path containing x’s declaration must contain a definite first use of x before the declaration or initialization of another local variable, and that definite first use must be either to pass x to another function’s out parameter, or else be an assignment-expression to x (which can be from a single-object ex-pression or a possibly-empty initializer_list) that performs a single constructor call (usually direct-initialization) of x with the right-hand side expression treated as the argument to x’s constructor.
If two local variables x and y are declared in that order and are explicitly uninitialized, then on every path x’s definite first use must precede y’s definite first use.

And I'll add this Note:

The requirement to initialize in declaration order preserves that variable lifetimes continue to nest correctly, so that a later-declared local variable y can safely refer to a previously-declared variable x that will outlive y by construction.

I'll be in the next version of the PDF I push. Thanks!

thanks that clarifies a lot. I'm a bit concerned about that limitation but that's a separate issue so I'll raise it separately once the paper is updated.

thanks that clarifies a lot. I'm a bit concerned about that limitation but that's a separate issue so I'll raise it separately once the paper is updated.


chore: update babel

This is a tooling update with the primary goal to speed things up during development, also allowing for easier debugging.
In the process support for IE and Node 6 has been dropped, allowing for async / await usage without transpilation.
The changes here are published as 3.0.0-alpha.0 and can be installed via
@patternplate/cli@next

Please test and feedback
cc @LKuechler @MartinHelmut @rgaertner

Obsolete via #263


default user agent is added in recursive url loader header

<!-- Thank you for contributing to LangChain!

Replace this entire comment with:
  - **Description:** Experiencing a 403 Forbidden error when using the RecursiveUrlLoader on certain websites, and that this issue is resolved when we manually set a User-Agent in the headers. The WebBaseLoader in LangChain has a default User-Agent set in the session headers but RecursiveUrlLoader doesn't have default User-Agent. Updated  RecursiveUrlLoader code to include default user-agent, if it is not provided.
  - **Issue:** This fixes 403 Forbidden error on certain websites. This code update will solve the issue mentioned in the link https://github.com/langchain-ai/langchain/issues/11540,https://github.com/langchain-ai/langchain/issues/11541.
  - **Dependencies:** fake_useragent library is needed,
  - **Tag maintainer:** @baskaryan, @eyurtsev, @hwchase17,

thanks for contribution @HiDhineshRaja, could you provide some context/motivation for the PR?


As a user, I want to see an error message when I hit a nonexistent page so that I know what went wrong.

Notes for testing
Edit the location in your browser to access a non-existent url. You should see a 404 page with a picture of Claus.

I did indeed see an error message and a lovely picture of Claus.


Error to install with PIP

# pip --version
pip 21.3.1 from /usr/local/lib/python3.8/site-packages/pip (python 3.8)

# pip install https://github.com/matplotlib/basemap/archive/master.zip#egg=basemap
Collecting basemap
  Downloading https://github.com/matplotlib/basemap/archive/master.zip (129.7 MB)
     |████████████████████████████████| 129.7 MB 10 kB/s
ERROR: basemap from https://github.com/matplotlib/basemap/archive/master.zip#egg=basemap does not appear to be a Python project: neither 'setup.py' nor 'pyproject.toml' found.

Which operating system are you using? If it is GNU/Linux (x86 o x64), you can use the precompiled binaries available in PyPI:
python -m pip install "basemap == 1.3.0b1" "basemap-data == 1.3.0b1"

It is the same problem as in #525, the direct pip installation from the GitHub repo is not working anymore because I have been restructuring the project folder structure in order to make the wheel packaging easier.
You would only need to install explicitly basemap-data until we have a stable release of 1.3.0 in PyPI, at that point basemap will install basemap-data automatically as dependency. If you need the high resolution datasets, you can install them from basemap-data-hires.


sorting in filemanager is broken

dateModified: in MongoDB, some dateModified data is showing up as strings, instead of date type. This only occurs for ones which haven't ever been saved after initial creation. to fix,

Aggregation to transform all dateModified from string to date type.
Upon creation of transcription, make sure dateModified gets added as date type.
Same should probably apply to dateCreated

alternatively, could just keep everything as strings.


Enable tests in CodingStyle directory

Let's include in the test suite the tests introduced in #4157

Tabs alert :)

Tabs alert :)

Well, even if it's a good start, we could not only test for tabs: there are a lot of style checks that could be done.
Maybe @KorvinSzanto has already done some tests about coding style checks?

I would really rather not have tests fail if coding standards aren't met. But the short tag check is good.

I swear to god I replied to this thread haha. I think we should do this with csfixer or phpmd. We can just add that as a script in .travis like i did with this project here: https://github.com/buttress/browserslist/blob/master/.travis.yml#L32
I definitely don't think it's worth the cpu to do this ourselves in phpunit so I'd like to update this to use something that is purpose made.

Well, style handling in view files (mix of php and html) is a nightmare.
I guess that readable view files won't pass coding style checks, and view files that pass coding style checks aren't readable that much (see for instance the mess introduced in view files by php-cs-fixer).
So, we have 2 options:

include all the files in the checks -> they won't pass or we'd need messy view files
exclude view files from tests -> short tags may and un in view files

I swear to god I replied to this thread haha

You replied here: https://github.com/concrete5/concrete5/pull/4380

@mlocati this is what I have in my scrutinizer branch right now. It only runs on one machine:
$ concrete/vendor/bin/phpcs -p --standard=psr2 --sniffs=Generic.PHP.DisallowShortOpenTagSniff,Generic.WhiteSpace.DisallowTabIndent concrete/src concrete/controllers

This works great with #4419 merged

My manual and stupid (but effective) check shows a red flag on tests for every php file.
Is this possible with your approach (not limited to controllers or src  directories)?

Yeah, just use
$ concrete/vendor/bin/phpcs -p --standard=psr2 --sniffs=Generic.PHP.DisallowShortOpenTagSniff,Generic.WhiteSpace.DisallowTabIndent concrete

But there are quite a lot of tab vs space issues in view files and I figured it's not really worth it anyway. IMO better get the classes solid than have overly permissive code style checks.


feat: Add is_at_start and is_at_end methods

Fixes #1018.
This is still a work in progress. Looking for help/feedback with the tests.
For this to work reliably and as expected, I believe that the Scroll object needs to "write back" to this ScrollbarState object, so that this ScrollbarState understands its viewport size. This communication is at the heart of most of the scrollbar deficiencies, as far as I can tell.

I think this could be closed for the reasons stated in #1018? There should be focus on improving the design, therefore #174.

Hmm it's been a very long time since I've thought of this library, and I ended up putting my project that was going to use it on hold until it was a bit more feature-full.
I'm happy if you think the best option is to close this, with a better alternative available (or soon-to-be-available) in the future.

with a better alternative available (or soon-to-be-available)

Currently, there isn't.

Marked as a draft for now. We can come back and investigate this at some later point if still needed.


Add CreateFromJson POST operation to ScaleUnit

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

Automation for azure-sdk-for-go

Automation for azure-sdk-for-js

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-python

Automation for azure-sdk-for-java

Can one of the admins verify this patch?

Automation for azure-sdk-for-net
A PR has been created for you:
https://github.com/azure/azure-sdk-for-net/pull/6243
.NET SDK Commits:
https://github.com/adxsdknet/azure-sdk-for-net/pull/6243/commits/ecaee494ff72e8b71c30774e08d8ee5b7097c4fc

adding @bganapa for review

@salameer Could you please help assigning this to one of the Spec reviewers? It looks like it did not auto-assign. Thanks!

@sphibbs Could you please assign to one of the spec reviewers? It did not auto assign for some reason. Thanks!


Dev Spaces with production cluster?

There seems to me conflicting information on using Dev Spaces on a production cluster. Early in the documentation under "Overview" and "About Azure Dev Spaces" it states "...is not intended to run on production AKS clusters". Then under "How to" and "Using CI/CD with Azure Dev Spaces" there is an actual example on how to structure namespaces on a production cluster with Dev Spaces.
While this deals with the content of documentation, and could probably be posted on another repository, I would like to get clarification on the technical implications on using Dev Spaces on a production cluster.
I understand that during development, using Dev Spaces, the containers are built using the active AKS cluster? As well there are a few pods required for Dev Spaces instrumentation? What other considerations/implications are there?

https://docs.microsoft.com/en-us/azure/dev-spaces/about
https://docs.microsoft.com/en-us/azure/dev-spaces/how-to/setup-cicd#dev-spaces-instrumentation-in-production

Hi David,
Dev Spaces is very much a development tool, and is not intended for use in production. What the documentation describes is how you can use Dev Spaces in a non-production cluster, or in a cluster that uses namespaces to separate between non-production and production workloads. The latter is not something we would recommend for isolation reasons, but is possible and may work well for smaller projects where cost is a concern.
The things to think about when enabling Dev Spaces on a production cluster would be:

The extra workloads that are deployed into the “azds” namespace, and the compute required to run these (minimal, but not nothing);
The extra overhead of pods that were deployed using Dev Spaces, where compute is used to build the container image that will be run, and the container image is typically fatter than an optimized production image;
The extra overhead of sidecar containers in namespaces that are designated as Dev Spaces (also rather minimal, but contributes to overall load).

Then of course the security implications of allowing developers to play in the same cluster as production workloads, especially given the fact that today, Dev Spaces effectively gives developers admin access to the cluster so they might inadvertently mess up the production workloads.
Hope that helps,
Stephen
From: David Hovey<EMAIL_ADDRESS>Date: Saturday, March 21, 2020 at 3:05 PM
To: Azure/dev-spaces<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [Azure/dev-spaces] Dev Spaces with production cluster? (#295)
There seems to me conflicting information on whether to use Dev Spaces on a production cluster. Early in the documentation under "Overview" and "About Azure Dev Spaces" it states "...is not intended to run on production AKS clusters". Then under "How to" and "Using CI/CD with Azure Dev Spaces" there is an actual example on how to structure namespaces on a production cluster with Dev Spaces.
While this deals with the content of documentation, and could probably be posted on another repository, I would like to get clarification on the technical implications on using Dev Spaces on a production cluster.
I understand that during development, using Dev Spaces, the containers are built using the active AKS cluster? As well there are a few pods required for Dev Spaces instrumentation? What other considerations/implications are there?

https://docs.microsoft.com/en-us/azure/dev-spaces/abouthttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdocs.microsoft.com%2Fen-us%2Fazure%2Fdev-spaces%2Fabout&data=02|01|stephpr%40microsoft.com|2e3733cab91342caa5bc08d7cde3f988|72f988bf86f141af91ab2d7cd011db47|1|0|637204251335702637&sdata=bfD2RG73l1OHwAiX%2FEu9HTY7mKbDNUqwu%2BRq9PhVTpw%3D&reserved=0
https://docs.microsoft.com/en-us/azure/dev-spaces/how-to/setup-cicd#dev-spaces-instrumentation-in-productionhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdocs.microsoft.com%2Fen-us%2Fazure%2Fdev-spaces%2Fhow-to%2Fsetup-cicd%23dev-spaces-instrumentation-in-production&data=02|01|stephpr%40microsoft.com|2e3733cab91342caa5bc08d7cde3f988|72f988bf86f141af91ab2d7cd011db47|1|0|637204251335712635&sdata=mz%2B0xr9nNHVo0w68VVUUlpSfxp3XC4cJ6WLQWs1dmJQ%3D&reserved=0

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fdev-spaces%2Fissues%2F295&data=02|01|stephpr%40microsoft.com|2e3733cab91342caa5bc08d7cde3f988|72f988bf86f141af91ab2d7cd011db47|1|0|637204251335712635&sdata=JVqAQs311p6voPt1gaLNCjWiJ8nzStpftx2pf%2BOmQL4%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAAHKYIWWZOB6WA4OS7QYMY3RIU22ZANCNFSM4LRCB4IQ&data=02|01|stephpr%40microsoft.com|2e3733cab91342caa5bc08d7cde3f988|72f988bf86f141af91ab2d7cd011db47|1|0|637204251335722625&sdata=iiIYPycbWjDek5UP1sjnlS3aw9L8ugcg2iaRciqAv20%3D&reserved=0.

This does help. Thanks.


Add handling of composite types (InputArrayOfArrays etc.)

This allows to generate bindings for some additional functions (for me it's necessary to use findContours).

Hi, thanks for your interest in this thing.
I managed to fix the travis build on master. Would you like to rebase against it to see if your PR builds ?

Yes, sure.

here we go, 0.2.4 released, thanks :)


Need a way to unregister and unsubscribe.

In the API there are calls to register a context calculator and calls to subscribe to events. I need a way to unsubscribe and unregister as well so my plugin works well with loading and unloading. Otherwise I cannot say that my plugin handles memory management properly.
A lot of people say you should never reload plugins. Personally, I reload my plugins individually 100 times before a server reboot is considered because of good housekeeping practices. To prevent any memory leaks I would like to have some unregister and unsubscribe API calls be added to luckperms. Can we make this happen? If so I would be very grateful. Thank you.

Based on what I have seen so far from the code I believe the easiest way to add this functionality without breaking existing API calls would be to make your system like the bukkit scheduler system. When you add a listener or something of that nature, return an integer key that can be used to remove the item later. Alternatively, allow people to unregister all "things" that have been registered by plugin X.

I tried to get the project up and running on my local computer, but was unable to resolve some of the issues with the project. If you're willing to help me get this thing set up I can take it the rest of the way and do these code changes for you.

Unregistering event handlers
Calling EventBus#subscribe returns a EventHandler instance. You can call EventHandler#unregister to unsubscribe.
Alternatively, you can use the subscribe method which accepts a plugin parameter. This will automatically unregister the handler when your plugin disables. (in the same way the Bukkit event system works)
Unregistering context calculators
(this was only added in the commit referenced above)
Use ContextManager#unregisterCalculator.

@lucko have you tested that fix on bungee cord?

I haven't tested but it should be absolutely fine.

One of the unregister methods as a blank method body. Is that intentional? I noticed that the register method in that same class is not left blank.

Can you send a link to that class?

Also I do not use sponge API but what I was browsing the code for that I noticed the register method would wrap the inserted context in a proxied sponge context wrapper or something like that.

That's correct, but the Sponge API doesn't offer a way to unregister calculators anyway.

https://github.com/lucko/LuckPerms/blob/85d2f0b51f7378fcf024046c5c1c4ca9e163bfbb/common/src/main/java/me/lucko/luckperms/common/api/implementation/ApiContextManager.java

Woops - thanks!

Thanks for fixing it man!


ADD: Getting started guide with docker instructions

closes: #95

@mgrover1 Shall we restructure a bit?
Moving Data Access and Notebook Environment into the Getting Started-Section?


samlocal compatibility with sam cli v1.15.0

Hi,
I followed the aws sam installation on linux to use brew to install sam cli.
I had followed the instruction at AWS
sam --version
SAM CLI, version 1.15.0

Is there a dependency and a working version of sam cli required for this to work.
Please advice.
Thank you
pip install aws-sam-cli-local Collecting aws-sam-cli-local   Using cached aws-sam-cli-local-<IP_ADDRESS>.tar.gz (2.7 kB) ERROR: Could not find a version that satisfies the requirement aws-sam-cli>=1.1.0 (from aws-sam-cli-local) (from versions: 0.3.0, 0.4.0, 0.5.0, 0.6.0, 0.6.1, 0.6.2, 0.7.0, 0.8.0, 0.8.1, 0.9.0, 0.10.0, 0.11.0, 0.12.0, 0.13.0, 0.14.0, 0.14.1, 0.14.2, 0.15.0, 0.16.0, 0.16.1, 0.17.0, 0.18.0, 0.19.0, 0.20.0, 0.20.1, 0.21.0, 0.22.0, 0.23.0) ERROR: No matching distribution found for aws-sam-cli>=1.1.0 (from aws-sam-cli-local)

Looks like the pip didnt have visibility of sam cli because it was installed as sudo. I was able to install it when I did the install as sudo.

Looks like the pip didnt have visibility of sam cli because it was installed as sudo. I was able to install it when I did the install as sudo.

Looks like the pip didnt have visibility of sam cli because it was installed as sudo. I was able to install it when I did the install as sudo.

Looks like the pip didnt have visibility of sam cli because it was installed as sudo. I was able to install it when I did the install as sudo.


Only update specific apps

As an alternative to the 'excluded_apps.txt' it would be nice to have a 'included_apps.txt' for whitelisting purpose, i.e when specifik apps are installed via SCCM that list could be updated and WAU should only update these, not anything else.

Hi.
Thanks for your comment. It was in my mind, so thanks to have written it down.
Just thinking about the best way to have white and black list options in install script.

My take on this is:
If there exist a white list (populated or unpopulated) then nothing gets updated until the white list gets populated.
I'm saving all our apps that we let the users manage as Available in SCCM in registry as Detection Method under HKEY_LOCAL_MACHINE\SOFTWARE\Winget
InstEd.InstEd=Installed
TimKosse.FileZilla.Client=Uninstalled
etc.
If/when there's a whitelist for WAU I'm gonna populate the list from the registry when users install/uninstall an app from SCCM

v1.7.0 published as pre-release for testing.
You can now use white list to only maintain app you want.
Juste create an included_apps.txt next to excluded_apps.txt, add the apps in list and run ".\Winget-AutoUpdate-Install.ps1 -UseWhiteList"

The idea if you use White List is :
if a file exists, keep it
if no file, create empty one
And if you use Winget-Install, you can install apps and use the "-WAUWhiteList" switch to add the app to white list when it's installed

Wonderful!
Used your WAU (stripped version) for install/repair (update)/uninstall single apps in SCCM before (because of the non existing whitelist feature in WAU.
In that way users could manually update single apps from Software Center.
Going to abandon my own additions and begin using your Winget-Install + WAU and $WAUWhiteList instead!
Thanks!

glad it helped ;)

Need some help with the whitelist file. I'm installing this via Intune and I wrapped my whitelist file inside the win32 app file. I seem to be getting this error. How should I fix existing devices and my new deployments via Intune.

Hi!
Don't include ‐Listpath in your install. It will find the list itself (default).

Ok so for those that were deployed with the ‐Listpath is there a way to fix them after the fact?

Yes, simply redeploy WAU.

Ok did that and same issue.

Does this look like it's working?

My folder contents:

Strange then...
...there's the string of dashes (when there's not the message of An unusual... pops up).
Are those apps in your white list?

No this is my whitelist.
Google.Chrome
AgileBits.1Password
Perimeter81Ltd.Perimeter81Ltd
SlackTechnologies.Slack
Zoom.Zoom

Well, run the same command as before as system in a terminal (psexec or nirsoft advanced run) and post the output.

https://www.nirsoft.net/utils/advanced_run.html

I agree, in your case we need to see what winget returns in system context.

Or/and the log from after ($upgradeResult is transferred to the log):


[Feature]: Expose internal "Presence" for general use

Discussed in https://github.com/radix-vue/radix-vue/discussions/963

Originally posted by sondh0127 May 27, 2024
Export internal "Presence" for general use
There are some use-cases for building open-hide animations that are best suited for Presence and data-state
<Presence :present="open">

    Content

</Presence>
```

Released on reka-ui@latest 😁


OAuth okta authentication

The okta provider currently only supports API tokens. These tokens have god-like permissions so sharing them with e.g backstage installations potentially carries a high level of risk. This change allows provisioning an OAuth app in okta that has the bare minimum scopes namely okta.groups.read and okta.users.read.
:heavy_check_mark: Checklist

[x] Added tests for new functionality and regression tests for bug fixes
[x] Added changeset (run yarn changeset in the root)
[ ] Screenshots of before and after attached (for UI changes)
[x] Added or updated documentation (if applicable)

I came here while trying to see if the plugin implemented the scoped OAuth 2.0 access tokens  supported by the okta-sdk-nodejs, so super happy to see this PR.
Evaluating the plugin ATM but will test your branch @jgrigg . Thanks!

@ocraviotto did you have any luck testing this branch? I have been a bit busy but might try again pushing it to our dev env today.

@ocraviotto did you have any luck testing this branch? I have been a bit busy but might try again pushing it to our dev env today.

I tested it but only with a PEM key as file input, like so:
catalog:
  rules:
    - allow: [User, Group]
      locations:
        - type: url
          exact: okta-org:all
  providers:
    okta:
      - orgUrl: 'https://[tenant].okta.com'
        oauth:
          clientId:
            $env: OKTA_OAUTH2_CLIENT_ID
          keyId:
            $env: OKTA_OAUTH2_KEY_ID
          privateKey:
            $file: ${OKTA_OAUTH2_PRIVATEKEY_PATH}

It pulled users and groups no problem, but I have not been able to test further (pass the PWK in JSON etc.)
So thanks!

Thanks, looks good to me! @ocraviotto thank you for the review, and do you have any more thoughts?

Just a question on braces in the example config, other than that, :+1:

Thanks, yea the readme can be fixed later! :) Thank you for the contribution, let's ship it!

Amazing thank you both :)


[STORE] allowed the updated_at timestamp to persist to the database upon...

Because the updated_at timestamping happens after the save, self[:updated_at] is only applying an updated_at timestamp to the model in memory, and not actually persisting that timestamp to the database. So it seems that the order of these lines needs to be flipped, with the gateway.save happening second, which will actually persist the updated_at change to the database. This would be more consistent with the update and touch methods which do actually persist the change.

Ah, thanks for the catch! Can you add a test for this? I think just adding another integration test is fine in this case.

Hi @kaptron, just pinging if you had time to add the tests? No time pressure, just curious :)

@karmi Have not yet -- but I should! Will try to get on top of that.

@kaptron Take your time, just ping me when you have something please!

I took a crack at this in #389

@kaptron I think I'll go with @ryansch's patch in #389, if that's OK.

Yep, sorry again for neglecting -- @ryansch's patch looks great!

Thanks guys, merged the patch!


stream_data_wifi.py "failed to parse" errors

Apologies in advance if this is not the right way to post such an issue.
When trying to run stream_data_wifi.py, I am able to connect to the board but the packets are garbled, as shown in the output below:
Welcome to OpenBCI Native WiFi Shield Driver - Please contribute code!
Opened socket on <IP_ADDRESS>:49624
Try to find WiFi shields on your local wireless network
Scanning for 5 seconds nearby devices...
Found WiFi Shield OpenBCI-4AAD with IP Address <IP_ADDRESS>
Init WiFi connection with IP: <IP_ADDRESS>
Connected to cyton with 8 channels
WiFi Shield to Python TCP Socket Established
WiFi Shield Instantiated
Incoming connection from ('<IP_ADDRESS>', 5672)
failed to parse: ?D??U??V?͆?? ?????s???F??#??9??k???u?????s???G????????????um?u??T?s,??H?????????Ú?u?????{?sU??I?????P?̍??%?u?%????r???J??{????????f?uO?S??1???K?????????é?u?????j?sE??L?????????å?u?????Y?s;????M??????????Ë?uv?r??8?s?
No JSON object could be decoded
I have an OpenBCI Cyton and Wifi Shield, both with up-to-date firmware.  Streaming works with the OpenBCI GUI, and I was able to get the js example (getStreaming.js) to work.
I'm running on a Mac so I was unable to install bluepy, but all other requirements are at or above the required versions.
Any assistance would be greatly appreciated.

Hello,
If the OpenBCI GUI is working then it is probably something wrong with the python script. It will be difficult to debug on my side, without a Wifi Shield. Which version of the scripts are you using, the master branch? @aj-ptw Had been working actively on a "1.0.0" branch, that isthe most up to date version of the code, and you should give it a try if not already.

I was using the masters branch, but I tried the 1.0.0 branch and it works! I now get output that looks normal:
0
[0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0]
0
[-0.007191427909306039, 0.005549111133707897, -0.0071619683101139435, -0.007198960447187477, -0.1540045474773106, -0.0487470535930459, -0.14287948076480397, -0.012493216990615962]
1
[-0.007196121775641653, 0.0055543190901659835, -0.007166550417727281, -0.007203497851311904, -0.1540087943087571, -0.04875130042449241, -0.14288372759625048, -0.012497620284273658]
I had to modify the init.py file to not include the Ganglion code, because it uses bluepy, but this solves my issue! Thanks!

Thanks for the feedback... at some point we should merge this branch to master ^^
Using try/catch we should be able to detect the need for Ganglion-related libraries, I will leave the bug report opened as a reminder.


Anyway to initiate a 'Refresh' while simply viewing the "Mobile" ngrok.io view

Thanks so much for these quick releases. Every version has been adding more and more useful features.
The map seems to update fine on my computer but when I leave my home and view the ngrok.io view of the map, it stops updating unless I remotely control my computer and restart the app. Is this because the App is going into some kind of "standby" when the computer isn't in use and stops refreshing? Is there anyways to initiate a refresh while remote using Mobile besides remote desktop connection.

im wondering too. i dont see a button to refresh when i put the address on my phone

I seem to have created an issue #117 which is more or less a duplicate of this with the addition of initiating a coordinate change.


display crates downloading & compiling process

If we do git clone a fresh big project and then open it in vscode OR vim, cargo needs to download and compiling the dependencies first, this can be very long.
However, during this period, I feel there is no progress shown in interface, users may think vscode is not working or RA is not working.
Is it possible to add also some texts or progress percentage to tell the users that cargo is downloading and compiling?

Let's say docs.rs, the reproduce steps are:

git clone https://github.com/rust-lang/docs.rs/
cd docs.rs
code .  OR vim src/lib

Ok it looks like a coc-rust-analyer issue


fix for golang version

:nut_and_bolt: Description: What code changed, and why?
Fixes for Pipeline which fail because of Golang version not changed in docker images of a2-integration tests.

:chains: Related Resources
:+1: Definition of Done
:athletic_shoe: How to Build and Test the Change
:white_check_mark: Checklist
All PRs must tick these:

[ ] I have read the CONTRIBUTING document.
[ ] All commits signed-off for the Developer Certification of Origin.

With occasional exceptions, all PRs from Progress employees must tick these:

[ ] Is the code clear? (complicated code or lots of comments--subdivide and use well-named methods, meaningful variable names, etc.)
[ ] Consistency checked? (user notifications, user prompts, visual patterns, code patterns, variable names)
[ ] Repeated code blocks eliminated? (adapt and reuse existing components, blocks, functions, etc.)
[ ] Spelling, grammar, typos checked? (at a minimum use make spell in any component directory)
[ ] Code well-formatted? (indents, line breaks, etc. improve rather than hinder readability)

All PRs from Progress employees should tick these if appropriate:

[ ] Tests added/updated? (all new code needs new tests)
[ ] Docs added/updated? (all customer-facing changes)

Please add a note next to any checkbox above if you are NOT ticking it.
:camera: Screenshots, if applicable

This doesn't fix the pipeline, as rel-eng needs to release new docker image for a2-integration, which is done in this PR 1793 of rel-eng.


naked file and image urls no longer download the file (causing SEO problems)

Describe the bug
Previous versions of plone /myimage.jpg would result in the image being viewed
Volto no longer works this way.
The major consequence seems to be search crawlers. The sitemap includes these naked urls. They get indexed and often counted as soft 404s or indexed as pages you really don't want visitors to find.
the robots.txt rules also make it very hard to protect these pages from indexing. the standard rules will preventing indexing /view pages for example, which are the same view
To Reproduce
add image. view the url
Expected behavior
I do think the old plone behaviour is better. but maybe there was a reason this was changed?
Screenshots
If applicable, add screenshots to help explain your problem.
Software (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Volto Version [e.g. 8.5.0]
Plone Version [e.g. 5.2.2]
Plone REST API Version [e.g. 7.0.1]

Additional context
Add any other context about the problem here.

Please describe the needed solution. Which URL is causing the problem and what image should be shown there instead of a 404 error?

@Programmer-Dost I didn't say there was a 404.
Plone classic will display /myimage as the raw image
Volto will render /myimage as a page which is the same as /myimage/view.
The consequences are bad for SEO, esp because the /myimage is whats listed in the sitemap.


Opening up access via @auth rule fails in giving read access for unAuth role

How did you install the Amplify CLI?
npm -g
If applicable, what version of Node.js are you using?
v18.13.0
Amplify CLI Version
12.2.3
What operating system are you using?
Debian 12
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
No manual changes - using all auto-generated resolvers in this issue, for example.
Describe the bug
This issue may be related to #1760, it may even be the same issue.
I have tried to open up an API model for access by unAuth role by adding { allow: private, operations: [read] } to the model @auth section but I still get an exception when trying to list projects, which says:
Query failed: UnknownException {
  "message": "unable to send GraphQLRequest to client.",
  "underlyingException": "SignedOutException {\n  \"message\": \"No user is currently signed in\"\n}"
}

I have verified first that the listProjects() query works when a user is the owner of the projects and is signed in. But when nobody is signed in it should still work with the above permissions but doesn't.
Here's the model for which I tried to list all instances:
type Project
  @model(subscriptions: { level: off })
  @auth(
    rules: [
      { allow: owner }
      # allow anyone not signed-in can read this model - we will add a filter
      # that only lets public model instances through, in the resolver code
      { allow: public, provider: iam, operations: [read] }
      # allow anyone signed-in can read this model - we will add a filter
      # that only lets public model instances through, in the resolver code
      { allow: private, operations: [read] }
    ]
  ) {
  id: ID!
  owner: ID @index(name: "projectsByOwner", sortKeyFields: ["updatedAt"])
            @auth(rules: [
              { allow: owner, operations: [create, read, delete] }
              { allow: public, provider: iam, operations: [read] }
              { allow: private, operations: [read] }
            ])
  user: User! @belongsTo(fields: ["owner"])
  title: String!
  visibility: ProjectVisibility!
  license: ProjectLicense!
  description: String
  artist: String
  coverartUrl: String
  parentProjectId: ID @index(name: "projectsByParentProject", sortKeyFields: ["updatedAt"])
  forks: [Project] @hasMany(indexName: "projectsByParentProject", fields: ["id"])
  nPlays: Int
  nTracks: Int
  nForks: Int
  duration: Float
  tracks: [Track] @hasMany(indexName: "tracksByProject", fields: ["id"])
  createdAt: AWSDateTime!
  updatedAt: AWSDateTime!
}

and here is the listProjects() code:
  Future<List<Project>> listProjects() async {
    log('listProjects()');
    try {
      final request = ModelQueries.list(Project.classType);
      final response = await Amplify.API.query(request: request).response;

      final projects = response.data?.items;
      if (projects == null) {
        log('errors: ${response.errors}');
        return const [];
      }
      List<Project> returnedProjects = [];
      for (var project in projects) {
        if (project != null) returnedProjects.add(project);
      }
      return returnedProjects;
    } on ApiException catch (e) {
      log('Query failed: $e');
      rethrow;
    }
  }

Expected behavior
After adding the @auth rule above I expected unAuth role to be able to list projects, i.e, I expected to be able to successfully execute the listProjects query with some results even when no user has yet signed into the application.
Reproduction steps

Use a similar schema model, perhaps a simplified one - the important thing is to use the same @auth rules
Create a few instances of the model by some user
Try to list all model instances while nobody is signed in

Project Identifier
860b46330dca368b2540f2cdb87a3485
Log output

# Put your logs below this line

Additional information
No response
Before submitting, please confirm:

[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

Hi @AnilMaktala - just trying to provide more info here: I noticed this other issue that may be related here. Please keep in mind that currently the default authorization mode is Cognito and also that the search table queries do work. Also: many other queries have worked for me with this schema and with Cognito as the default authorization mode. This issue is for the case when an unauth IAM user tries to do a listProjects query. Thanks!

@AnilMaktala the following is quite interesting / related: I have been trying to "open up" the above schema so that I can continue working on my project, by removing all field level authorization rules and by providing, in the same schema, the rules that open every operation to public and private and owner (except that I did not touch the Search model because it was not related) -- after making all of these permissions changes and opening things up, the same exact issue remains:
Query failed: UnknownException {
"message": "unable to send GraphQLRequest to client.",
"underlyingException": "SignedOutException {\n  "message": "No user is currently signed in"\n}"
}
[ i.e, same response to the listQueries() call shown above. ]
Here is the full schema that was just "opened up" to give more permissions which also does not work:
[ Note that the amplifyconfiguration.dart shows that Cognito is the default auth mode for the GraphQL endpoint. ]
type Search
  @model(subscriptions: { level: off })
  @auth(rules: [
    { allow: public, provider: iam, operations: [create, read, update] }
    { allow: private, operations: [create, read, update] }
  ]) {
  normalizedQuery: String! @primaryKey
  titles: String!
  nTimes: Float!
  nTimesInLast24h: Int!
  titlesUpdatedAt: String!
  createdAt: AWSDateTime!
  updatedAt: AWSDateTime!
}

type Track
  @model(subscriptions: { level: off })
  @auth(rules: [
    { allow: owner }
    { allow: public, provider: iam }
    { allow: private }
  ]) {
  id: ID!
  owner: ID
  projectId: ID! @index(name: "tracksByProject", sortKeyFields: ["updatedAt"])
  duration: Float!
  uploadFilename: String!
  nPlays: Int!
  title: String!
  description: String
  imageUrl: String
  createdAt: AWSDateTime!
  updatedAt: AWSDateTime!
}

enum ProjectLicense {
  someRightsReserved
  by
  bySa
  byNd
  byNc
  byNcEu
  byNcSa
  byNcNdEu
  cc0
  publicDomainMark
}

enum ProjectVisibility {
  private
  public
}

type Project
  @model(subscriptions: { level: off })
  @auth(rules: [
    { allow: owner }
    { allow: public, provider: iam }
    { allow: private }
  ]) {
  id: ID!
  owner: ID @index(name: "projectsByOwner", sortKeyFields: ["updatedAt"])
  user: User! @belongsTo(fields: ["owner"])
  title: String!
  visibility: ProjectVisibility!
  license: ProjectLicense!
  description: String
  artist: String
  coverartUrl: String
  parentProjectId: ID @index(name: "projectsByParentProject", sortKeyFields: ["updatedAt"])
  forks: [Project] @hasMany(indexName: "projectsByParentProject", fields: ["id"])
  nPlays: Int
  nTracks: Int
  nForks: Int
  duration: Float
  tracks: [Track] @hasMany(indexName: "tracksByProject", fields: ["id"])
  createdAt: AWSDateTime!
  updatedAt: AWSDateTime!
}

enum PictureType {
  initials
  icon
  image
}

type User
  @model(subscriptions: { level: off })
  @auth(rules: [
    { allow: owner }
    { allow: public, provider: iam }
    { allow: private }
  ]) {
  id: ID!
  owner: ID
  displayName: String!
  colorIndex: Int!
  pictureType: PictureType!
  initials: String!
  about: String!
  place: String!
  nPlays: Int!
  nProjects: Int!
  nArrangements: Int!
  nPlaylists: Int!
  bytesQuota: Int!
  bytesUsed: Int!
  email: String!
  nUpdatesLeft: Int!
  nUploadsLeft: Int!
  totalEarned: Float!
  lastUploadedAt: AWSDateTime!
  projects: [Project] @hasMany(indexName: "projectsByOwner", fields: ["owner"])
  createdAt: AWSDateTime!
  updatedAt: AWSDateTime!
}

@AnilMaktala I have done one more test to try and open up the authentication restrictions so that I can continue to work with the UX, but even that new test did not succeed. It is illuminating, however, so I'll describe it. In short, removed all authentication code from the VTL resolver -- still, a IAM (unAuth role) cannot listProjects - with this same issue ...
There are 4 resolvers associated with Query.listProjects:
> ls amplify/backend/api/tracktunes/build/resolvers/Query.listProjects.*
amplify/backend/api/tracktunes/build/resolvers/Query.listProjects.auth.1.req.vtl
amplify/backend/api/tracktunes/build/resolvers/Query.listProjects.postAuth.1.req.vtl
amplify/backend/api/tracktunes/build/resolvers/Query.listProjects.req.vtl
amplify/backend/api/tracktunes/build/resolvers/Query.listProjects.res.vtl

I modified the contents of Query.listProjects.auth.1.req.vtl to remove authentication and put the new file in
amplify/backend/api/tracktunes/resolvers/Query.listProjects.auth.1.req.vtl

The new code in that file is just this:
## [Start] Authorization Steps. **
$util.qr($ctx.stash.put("hasAuth", true))
#set( $isAuthorized = true )
#set( $primaryFieldMap = {} )
$util.toJson({"version":"2018-05-29","payload":{}})
## [End] Authorization Steps. **

Still, I get the exact same exception thrown as described above!
This suggests that the problem is probably not with the VTL code, it probably is in the Dart Amplify library code.

@AnilMaktala I solved the issue and it is no longer an issue! Here's the summary of the solution:
This was not a problem with any VTL resolver code.
This was not a problem with @auth directives in the schema - they were correct.
It was a wrong setup problem with amplifyconfiguration.dart - a new API block has to be created in amplifyconfiguration.dart when using multi-auth -- just copy the block in there with authorizationType: AMAZON_COGNITO_USER_POOLS into a new block in the same API section and change just the authorizationType value in the new block to be "authorizationType": "AWS_IAM", and also change the name of the entire block at the top of it to something different (in this case it was changed to tracktunesGuest). Here are the changes from the OLD section in amplifyconfiguration.dart:
            "awsAPIPlugin": {
                "tracktunes": {
                    "endpointType": "GraphQL",
                    "endpoint": "https://sbtbwxoahfhfviwomoz3eg5t2e.appsync-api.us-east-1.amazonaws.com/graphql",
                    "region": "us-east-1",
                    "authorizationType": "AMAZON_COGNITO_USER_POOLS"
                },

to:
            "awsAPIPlugin": {
                "tracktunes": {
                    "endpointType": "GraphQL",
                    "endpoint": "https://sbtbwxoahfhfviwomoz3eg5t2e.appsync-api.us-east-1.amazonaws.com/graphql",
                    "region": "us-east-1",
                    "authorizationType": "AMAZON_COGNITO_USER_POOLS"
                },
                "tracktunesGuest": {
                    "endpointType": "GraphQL",
                    "endpoint": "https://sbtbwxoahfhfviwomoz3eg5t2e.appsync-api.us-east-1.amazonaws.com/graphql",
                    "region": "us-east-1",
                    "authorizationType": "AWS_IAM"
                },

Then in your code instead of using
final request = ModelQueries.list(Project.classType);

as in the code above, change it to:
 final request = ModelQueries.list(Project.classType,
          authorizationMode: APIAuthorizationType.iam,
          apiName: 'tracktunesGuest');

And now multi-auth is working allowing guests to read while giving more permissions to signed in users.
This blog helped me solve this issue - extending a thank you to FullstackPho!

Thank you @dorontal for the great write-up, this just saved me HOURS of work!! 🙌

@dorontal @tiyberius Glad you found my blog post useful and happy it is still (somewhat?) relevant almost 4 years after writing it! Thanks so much for the shoutout, really makes it worth it! :)


The link of the weights unavailable

Thanks for your information. Just update the link and it should work now.