Fix the version number string cannot be found in the dynamic library of boringssl.

fix  #678

before
2024-12-02T14:36:02Z INF AppName="eCapture(旁观者)"
2024-12-02T14:36:02Z INF HomePage=https://ecapture.cc
2024-12-02T14:36:02Z INF Repository=https://github.com/gojue/ecapture
2024-12-02T14:36:02Z INF Author="CFC4N<EMAIL_ADDRESS>2024-12-02T14:36:02Z INF Description="Capturing SSL/TLS plaintext without a CA certificate using eBPF. Supported on Linux/Android kernels for amd64/arm64."
2024-12-02T14:36:02Z INF Version=androidgki_arm64:v0.8.11:6.5.0-1025-azure
2024-12-02T14:36:02Z INF Listen=localhost:28256
2024-12-02T14:36:02Z INF eCapture running logs logger=
2024-12-02T14:36:02Z INF the file handler that receives the captured event eventCollector=
2024-12-02T14:36:02Z WRN ========== module starting. ==========
2024-12-02T14:36:02Z INF Kernel Info=5.15.41 Pid=5425
2024-12-02T14:36:02Z WRN Your environment is like a container. We won't be able to detect the BTF configuration.
If eCapture fails to run, try specifying the BTF mode. use `-b 2` to specify non-CORE mode.
2024-12-02T14:36:02Z INF BTF bytecode mode: CORE. btfMode=0
2024-12-02T14:36:02Z INF master key keylogger has been set. eBPFProgramType=Text keylogger=
2024-12-02T14:36:02Z INF module initialization. isReload=false moduleName=EBPFProbeOPENSSL
2024-12-02T14:36:02Z INF Module.Run()
2024-12-02T14:36:02Z INF listen=localhost:28256
2024-12-02T14:36:02Z INF https server starting...You can update the configuration file via the HTTP interface.
2024-12-02T14:36:02Z ERR OpenSSL/BoringSSL version check failed error="OpenSSL/BoringSSL version not found" soPath=/apex/com.android.conscrypt/lib64/libssl.so
2024-12-02T14:36:02Z INF setupManagers eBPFProgramType=Text
2024-12-02T14:36:02Z FTL module run failed. error="OpenSSL/BoringSSL version not found" isReload=false

after
emu64a:/data/local/tmp # ./ecapture tls
2024-12-02T15:03:52Z INF AppName="eCapture(旁观者)"
2024-12-02T15:03:52Z INF HomePage=https://ecapture.cc
2024-12-02T15:03:52Z INF Repository=https://github.com/gojue/ecapture
2024-12-02T15:03:52Z INF Author="CFC4N<EMAIL_ADDRESS>2024-12-02T15:03:52Z INF Description="Capturing SSL/TLS plaintext without a CA certificate using eBPF. Supported on Linux/Android kernels for amd64/arm64."
2024-12-02T15:03:52Z INF Version=androidgki_arm64:0.8.10-20241116-fcddaeb:5.15.0-125-generic
2024-12-02T15:03:52Z INF Listen=localhost:28256
2024-12-02T15:03:52Z INF eCapture running logs logger=
2024-12-02T15:03:52Z INF the file handler that receives the captured event eventCollector=
2024-12-02T15:03:52Z WRN ========== module starting. ==========
2024-12-02T15:03:52Z INF Kernel Info=5.15.41 Pid=6845
2024-12-02T15:03:52Z WRN Your environment is like a container. We won't be able to detect the BTF configuration.
If eCapture fails to run, try specifying the BTF mode. use `-b 2` to specify non-CORE mode.
2024-12-02T15:03:52Z INF BTF bytecode mode: CORE. btfMode=0
2024-12-02T15:03:52Z INF master key keylogger has been set. eBPFProgramType=Text keylogger=
2024-12-02T15:03:52Z INF listen=localhost:28256
2024-12-02T15:03:52Z INF https server starting...You can update the configuration file via the HTTP interface.
2024-12-02T15:03:52Z INF module initialization. isReload=false moduleName=EBPFProbeOPENSSL
2024-12-02T15:03:52Z INF Module.Run()
2024-12-02T15:03:52Z ERR bpfFile=boringssl_a_13_kern.o sslVersion=android_default
2024-12-02T15:03:52Z ERR OpenSSL/BoringSSL version not found, used default version.If you want to use the specific version, please set the sslVersion parameter with "--ssl_version='boringssl_a_13'" , "--ssl_version='boringssl_a_14'", or use "ecapture tls --help" for more help.
2024-12-02T15:03:52Z INF Hook masterKey function ElfType=2 Functions=["SSL_in_init"] binrayPath=/apex/com.android.conscrypt/lib64/libssl.so
2024-12-02T15:03:52Z INF target all process.
2024-12-02T15:03:52Z INF target all users.
2024-12-02T15:03:52Z INF setupManagers eBPFProgramType=Text
2024-12-02T15:03:52Z INF BPF bytecode file is matched. bpfFileName=user/bytecode/boringssl_a_13_kern_core.o
2024-12-02T15:03:52Z INF perfEventReader created mapSize(MB)=4
2024-12-02T15:03:52Z INF perfEventReader created mapSize(MB)=4
2024-12-02T15:03:52Z INF module started successfully. isReload=false moduleName=EBPFProbeOPENSSL
2024-12-02T15:04:01Z ??? UUID:844_6856_Thread-96_0_1_<IP_ADDRESS>, Name:HTTPRequest, Type:1, Length:212
GET /generate_204 HTTP/1.1
Host: www.google.com
Accept-Encoding: gzip
Connection: close
User-Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/60.0.3112.32 Safari/537.36

2024-12-02T15:04:01Z ??? UUID:844_6856_Thread-96_0_0_<IP_ADDRESS>, Name:HTTPResponse, Type:3, Length:184
HTTP/1.1 204 No Content
Connection: close
Alt-Svc: h3=":443"; ma=2592000,h3-29=":443"; ma=2592000
Cross-Origin-Resource-Policy: cross-origin
Date: Mon, 02 Dec 2024 15:04:00 GMT

2024-12-02T15:04:04Z ??? UUID:844_7597_Thread-99_0_1_<IP_ADDRESS>, Name:HTTPRequest, Type:1, Length:212
GET /generate_204 HTTP/1.1
Host: www.google.com
Accept-Encoding: gzip
Connection: close
User-Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/60.0.3112.32 Safari/537.36

2024-12-02T15:04:05Z ??? UUID:844_7597_Thread-99_0_0_<IP_ADDRESS>, Name:HTTPResponse, Type:3, Length:184
HTTP/1.1 204 No Content
Connection: close
Alt-Svc: h3=":443"; ma=2592000,h3-29=":443"; ma=2592000
Cross-Origin-Resource-Policy: cross-origin
Date: Mon, 02 Dec 2024 15:04:04 GMT

2024-12-02T15:04:06Z ??? UUID:844_7614_Thread-102_0_1_<IP_ADDRESS>, Name:HTTPRequest, Type:1, Length:212
GET /generate_204 HTTP/1.1
Host: www.google.com
Accept-Encoding: gzip
Connection: close
User-Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/60.0.3112.32 Safari/537.36

2024-12-02T15:04:06Z ??? UUID:844_7614_Thread-102_0_0_<IP_ADDRESS>, Name:HTTPResponse, Type:3, Length:184
HTTP/1.1 204 No Content
Connection: close
Alt-Svc: h3=":443"; ma=2592000,h3-29=":443"; ma=2592000
Cross-Origin-Resource-Policy: cross-origin
Date: Mon, 02 Dec 2024 15:04:05 GMT

^C2024-12-02T15:04:07Z INF module close.
2024-12-02T15:04:07Z INF Module closed,message recived from Context
2024-12-02T15:04:08Z INF iModule module close
2024-12-02T15:04:08Z INF bye bye.
emu64a:/data/local/tmp # %                                                                                ```

Congratulations!This bug has been successfully solved, but there is a new bug waiting for you:
OP565FL1:/data/local/tmp $ ./ecapture -b 2 tls -m text
2024-12-03T13:07:07Z INF AppName="eCapture(旁观者)"
2024-12-03T13:07:07Z INF HomePage=https://ecapture.cc
2024-12-03T13:07:07Z INF Repository=https://github.com/gojue/ecapture
2024-12-03T13:07:07Z INF Author="CFC4N<EMAIL_ADDRESS>2024-12-03T13:07:07Z INF Description="Capturing SSL/TLS plaintext without a CA certificate using eBPF. Supported on Linux/Android kernels for amd64/arm64."
2024-12-03T13:07:07Z INF Version=androidgki_arm64:v0.8.12:6.5.0-1025-azure
2024-12-03T13:07:07Z INF Listen=localhost:28256
2024-12-03T13:07:07Z INF eCapture running logs logger=
2024-12-03T13:07:07Z INF the file handler that receives the captured event eventCollector=
2024-12-03T13:07:07Z WRN ========== module starting. ==========
2024-12-03T13:07:07Z INF Kernel Info=6.1.75 Pid=9672
2024-12-03T13:07:07Z INF BTF bytecode mode: non-CORE. btfMode=2
2024-12-03T13:07:07Z INF master key keylogger has been set. eBPFProgramType=Text keylogger=
2024-12-03T13:07:07Z INF module initialization. isReload=false moduleName=EBPFProbeOPENSSL
2024-12-03T13:07:07Z INF Module.Run()
2024-12-03T13:07:07Z ERR OpenSSL/BoringSSL version not found, used default version.If you want to use the specific version, please set the sslVersion parameter with "--ssl_version='boringssl_a_13'" , "--ssl_version='boringssl_a_14'", or use "ecapture tls --help" for more help.
2024-12-03T13:07:07Z ERR bpfFile=boringssl_a_13_kern.o sslVersion=android_default
2024-12-03T13:07:07Z INF Hook masterKey function ElfType=2 Functions=["SSL_in_init"] binrayPath=/apex/com.android.conscrypt/lib64/libssl.so
2024-12-03T13:07:07Z INF target all process.
2024-12-03T13:07:07Z INF target all users.
2024-12-03T13:07:07Z INF setupManagers eBPFProgramType=Text
2024-12-03T13:07:07Z INF BPF bytecode file is matched. bpfFileName=user/bytecode/boringssl_a_13_kern_noncore.o
2024-12-03T13:07:07Z INF listen=localhost:28256
2024-12-03T13:07:07Z INF https server starting...You can update the configuration file via the HTTP interface.
2024-12-03T13:07:07Z FTL module run failed. error="couldn't init manager xxx error:operation not permitted , couldn't adjust RLIMIT_MEMLOCK" isReload=false


Improve error handling

Specifically this instance:
https://github.com/adriaandejonge/contentful-hugo/blob/9a944b96c7bc4eb0024ececd73a36105345f8a47/dump/types.go#L15-L23

Consider centralizing error handling. log.Fatal may work well for a command line utility but would be less user friendly for the Server Mode.


Incorrect steps

Install and configure Pacemaker on each cluster node under this sections, you have the following command:
sudo apt-get install -y pacemaker pacemaker-cli-utils crmsh resource-agents fence-agents corosync python3-azure
Then you have the following section: Enable and start pcsd service and Pacemaker with these:
sudo systemctl enable pcsd
sudo systemctl start pcsd
sudo systemctl enable pacemaker
Your installing crmsh in the first section command, but after that your telling us to start and enable pcsd, crmsh and has different command structure to pcsd.
You either need to change the command and remove crmsh and replace with pcs or you need to show the correct syntax to start crmsh and enable it.

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 9e89d1f9-5679-ffd4-8069-0d53756a41e6
Version Independent ID: 230ea633-ca5a-ab4d-f3b6-a6654afcd714
Content: Configure a Pacemaker cluster for SQL Server availability groups - SQL Server
Content Source: docs/linux/sql-server-linux-availability-group-cluster-pacemaker.md
Service: sql
Sub-service: linux
GitHub Login: @rwestMSFT
Microsoft Alias: randolphwest

copying @aravindmahadevan-ms for further action and FYI
Thank you for the feedback. @rwestMSFT could you please help remove the entire "enable and start pcsd service section". I am creating a PR with the required changes that needs to go in.

An internal PR has been created. Once it is merged, this issue will automatically close. Thanks again for your submission.


added new routes get and post at bubble-id/join, get shows the join p…

…age, posting to the route should allow new user to join
@jackrr Should I should try to reuse some of our existing login flow? In this case, I'll likely have to add arguments to your existing functions — primarily a bubble id, to indicate that a new member is intending to join a bubble in addition to being created.
Any thoughts or caveats on how to do the above?
Or should I create new function that handles a member who is logging in for the purpose of joining a specific bubble?

@dmahfouda I think the "right" way to go about it would be to change the app in such a way that the login code doesn't need to know about the "join" use case at all.
The way I'd build it is as follows:

Add a new /bubble/:bubbleId/join route that adds the currently logged-in user (via (:current-user req)) as a member to the bubble specified in the route param
Make the new handler from the above step redirect logged-out sessions to login (see login/logged-in usage for an example of a route that does this)
Improve the login flow so that it redirects back to the original route after successful login (this will lead to the user being added to the bubble via the default logic built in step 1)

This can be built by carrying a long a redirect query param through each step of the login flow

This way we can keep the login logic concerned only with login -- it doesn't need to worry about the manifold reasons why a user might need to login.
Does the above make sense? Thoughts?
I would be happy to pair on implementing this, though I'm confident you know enough to do steps 1 and 2 comfortably and at least get started on 3.


cpio required for 6.6 build

Windows Version
Microsoft Windows [Version 10.0.22631.3810]
WSL Version
WSL version: <IP_ADDRESS>
Are you using WSL 1 or WSL 2?

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>-2
Distro Version
Ubuntu 24.04
Other Software
No response
Repro Steps
Clone WSL2 kernel repo.  Install dependencies.
make -j$(nproc) KCONFIG_CONFIG=Microsoft/config-wsl

Expected Behavior
Kernel builds.
Actual Behavior
Build error
***
*** 'cpio' could not be found.
***

6.6 branch and the writeup here, https://learn.microsoft.com/en-us/community/content/wsl-user-msft-kernel-v6, should be updated to specify new dependencies for the 6.6 build.
sudo apt install cpio

Resolves the issue and 6.6 is built.
Thanks!
JP
Diagnostic Logs
I can include more make output if you'd like.

Logs are required for review from WSL team
If this a feature request, please reply with '/feature'. If this is a question, reply with '/question'. Otherwise please attach logs by following the instructions below, your issue will not be reviewed unless they are added. These logs will help us understand what is going on in your machine.
How to collect WSL logs
Download and execute collect-wsl-logs.ps1 in an administrative powershell prompt:
Invoke-WebRequest -UseBasicParsing "https://raw.githubusercontent.com/microsoft/WSL/master/diagnostics/collect-wsl-logs.ps1" -OutFile collect-wsl-logs.ps1
Set-ExecutionPolicy Bypass -Scope Process -Force
.\collect-wsl-logs.ps1

The scipt will output the path of the log file once done.
Once completed please upload the output files to this Github issue.
Click here for more info on logging If you choose to email these logs instead of attaching to the bug, please send them to<EMAIL_ADDRESS>with the number of the github issue in the subject, and in the message a link to your comment in the github issue and reply with '/emailed-logs'.

/question

@kelsey-steele - could you make sure that cpio is called out as a dependency in the readme? Thanks!
https://github.com/microsoft/WSL2-Linux-Kernel/blob/linux-msft-wsl-5.15.y/README.md

@kelsey-steele - could you make sure that cpio is called out as a dependency in the readme? Thanks!
https://github.com/microsoft/WSL2-Linux-Kernel/blob/linux-msft-wsl-5.15.y/README.md

Slight clarification; this branch: https://github.com/microsoft/WSL2-Linux-Kernel/blob/linux-msft-wsl-6.6.y/README.md

@benhillis @JPHutchins ah! yes. Can get that added before the next 6.6 kernel update. Will update here once done.
Thank you for the report and ping! :)

@kelsey-steele While I have you!  I'm curious how to test the releases that are on the Releases page?  Normally if I build a new kernel I can swap it out easily and keep my existing "Linux OS".  But the releases seem different, like I might not be able to keep my existing OS install?

@JPHutchins using a custom kernel is the same as before and shouldn't really be altering any of your WSL instances you have set up. Well altering outside what the kernel provides. The big difference is the inclusion of loadable modules with the 6.6 kernel while previously all modules were built in. At this time, loadable module support gets disabled when using a custom kernel which means a lot of functionality you're use to gets removed once you add that "kernel=" line into .wslconfig.
To get around this, at least until loadable module support is available with custom kernels, I recommend building the 6.6 WSL kernel with the last 5.15 configs. Or you can modify the config file to just build in everything you need, though taking the 5.15 config will likely be less work, especially if it already included everything you wanted.
Does this explain what you're seeing that looks to alter your WSL installs? If not, let's figure that out to make sure something wrong isn't occurring.

Does this explain what you're seeing that looks to alter your WSL installs? If not, let's figure that out to make sure something wrong isn't occurring.

There's nothing wrong!  If I'm fiddling with various kernels I just need to make sure I don't wipe out my Linux environment accidentally 🤣.  Thanks for the info!


gogetdoc should work even if the package layout is weird

For reasons I can't get into publicly, I'm working on Go code that doesn't conform to the standard package layout.
The line at https://github.com/zmb3/gogetdoc/blob/master/main.go#L110 causes the whole thing to bail in this case.
I understand that supporting weird package layouts isn't useful, but could there be some workaround so at least if you're trying to get to the standard libraries it still works?

@cflewis Perhaps there should hooks for a drop-in Go file with your environment specific stuff as is done with goimports in your environment.

I might be able to help you if you shared a little more info about your setup.  Supporting "weird layouts" could mean just about anything.  Can you elaborate?

I do have the same issue, the error I get is:
gogetdoc: couldn't get package for /home/$USER/git/go/lca2015/part1/src/main.go: can't find package containing /home/$USER/git/go/lca2015/part1/src/main.go`

@zmb3 any ideas why that happens? I'm obviously doing it the wrong way, but what structure does it expect?

@brunnre8 it expects a valid GOPATH, as outlined in https://golang.org/doc/code.html – and files must be in a package, $GOPATH/src/main.go is not valid, as it's not a package.

@dominikh thank you for the clarification.

(sorry for taking so long to loop back on this)
Elaboration: Bazel doesn't enforce the same code structure for Go as go build does. If I'm not using the go build Go structure (e.g. I don't even have a valid GOPATH) gogetdoc breaks. The format/existence of my own package shouldn't be a blocker to my ability to read the standard docs.

I do have somer other setups with symlinks (I'm used to put my projects in separate directory, outside of $GOPATH). go build and other go tools are working well with it, apart from gogetdoc.
Here is an example:
~/work/myproject
~/worspace-go/src/myproject -> ~/work/myproject
I can run go build and other gofmt in ~/work/myproject with success. However running gogetdoc says:
gogetdoc: couldn't get package for foo.go: can't find package containing ~/work/myproject/foo.go
Do you think support for this would be possible?

Many Go tools generally don't work well with symlinks in your GOPATH.
https://groups.google.com/forum/#!msg/golang-nuts/f5ZYztyHK5I/2Owd4vhdjXEJ
https://github.com/tools/godep/issues/71
https://github.com/joefitzgerald/go-plus/issues/159
https://github.com/golang/go/issues/11305
To answer your question, yes I think supporting this is possible. Whether or not that means we should do it is another story.  It seems to be a rare case, and I'd rather advocate the same message as other tools - don't use symlinks.

I have same issue. Moved my project to GOPATH to avoid it. Maybe it not that rare for people outside GOLANG community. This is the only language I'm using which forces me to keep my project in a specific place.

I don't know if I'm understanding this correctly, but I BELIEVE this means that to use gogetdoc I have to have my program in the GOPATH/src/ directory? Why is this enforced?
My workflow is currently ~/Code/[language]/[project]/[files] and all my other tools work fine. Is there a way to use gogetdoc on these files without moving my project into ~/.go/src/[project] ? I guess if I'm understanding this correctly I don't see why I would want an unfinished personal project in my GOPATH where I keep libraries.

@QantumEntangled you do need to have your program in $GOPATH/src.  I understand that this seems odd to newcomers and those familiar with other languages, but it’s the state of Go right now.  As @denispeplin said - it is the only language [that I’m aware of] that forces you to put your code in a certain place.  I personally hope to see this change, as it puts off a lot of folks.
As for why it’s enforced:  the go tool itself requires a GOPATH to build and install Go programs and run tests.  Linters and static analyzers require it to analyze your code.  Tools like gorename and guru require it in order to operate on your source code.  And lastly, the go/types package in the standard library (which gogetdoc is built on) requires it to type check your program.

I don’t see why I would want an unfinished personal project in my GOPATH where I keep libraries.

  GOPATH is just a directory structure - it does not require you to commit or otherwise publish your unfinished source code.


Allow HTML Temp AddOn shows old version

The HTML Temp AddOn shows the old version v7.0.6 for older Thunderbirds v91.0 - 100.0 on main page https://addons.thunderbird.net/de/thunderbird/addon/allow-html-temp/
It should be the newest v8.1.2 for newer Thunderbirds v100.0 - 107.0 (current release of Thunderbird is v102.3.1) as on versions page https://addons.thunderbird.net/de/thunderbird/addon/allow-html-temp/versions/
PS: The AddOn developer said that he has no influence related to which release version of the AddOn is displayed.

This is intended. If you view the website in Thunderbird, it will show the correct version. The website has no way to know what version of Thunderbird you are using, so it has a default, and that default is still 91.
This will be changed pretty soon now that 102 updates are going out quickly.

Oooook, thank you for clarification.

@Sancus clarified, but the change is yet to be made. However we don't need to reopen because some just created a new issue 199


Laravel 10 non-compatible examples

PHP & Platform
8.2.3
Database
MySQL 8
Laravel version
10.1.5
Have you done this?

[X] I have checked my logs and I'm sure is a bug in this package.
[X] I can reproduce this bug in isolation (vanilla Laravel install)
[X] I can suggest a workaround as a Pull Request

Expectation
I treid to make it work on Laravel 10 with Breeze installed but it's not working. Is it Laravel 10 compatible?
Description
The docs seem to be for older Laravel version.
For example I am getting this error when I try to add traits as in your docs: Undefined type 'Laragear\TwoFactor\Contracts\TwoFactorAuthenticatable'.intelephense(1009)
Reproduction
Also the implementation with routing and/or breeze etc. is missing in the docs as well.

By the way, is there any demo app or existing app where we can look how it should be implemented? I am especially interested in routes in the final app. But also final controllers.

Stack trace & logs
No response

I treid to make it work on Laravel 10 with Breeze installed but it's not working. Is it Laravel 10 compatible?

Yes

The docs seem to be for older Laravel version.

Only the links.

For example I am getting this error when I try to add traits as in your docs. Undefined type 'Laragear\TwoFactor\Contracts\TwoFactorAuthenticatable'.intelephense(1009)

If that was an error, the package would pass any test.

Also the implementation with routing and/or breeze etc. is missing in the docs as well.

Haven't using Breeze in years. You're free to PR with the instructions.

By the way, is there any demo app or existing app where we can look how it should be implemented?

I would but not my focus right now.


Uneditable background on hyprland

I write styles for my own waybar rice, there's some background behind waybar.

solved by adding
layerrule = ignorezero, waybar to hyprland config


Adds console resize support.

Resolves: #76

@jterry75,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

I have verified this through docker for both the init process and an exec process such as VI that the console resize is indeed happening.

What happens if the TTY relay finishes and closes master just as the resize console call is coming down?
It seems like some synchronization might be missing there.

I certainly could add synchronization but I didn't for two reasons.

If the resize is right before a close it will succeed. If the resize is right after a close you will get an error from the ioctl call which is no different than returning an error "invalid process" or whatever from locking and detecting the case. IE: the kernel is already a form of synchronization.
What is the case where you have a TTY exit the process and then resize it? The TTY case as far as I know is only for interactive sessions in which I am not physically fast enough to exit the process while resizing it.

All that said I am happy to add a mutex on the relay that exposes the console handle. Let me know if you still want it.

The kernel is not a safe form of synchronization. The fd can get reused right after the close call, so you will send the ioctl to the wrong place.
Why can't the relay implement the resize method? Then you can synchronize internally without exposing a mutex.

:clock1:

Anymore feedback here? Would like to get this checked in


Update styling of overflow tabs

This PR updates the styling of overflow tabs:

Render widget label in overflow menu.
Increased spacing for badge icons and tick icon (for active tab).
Widget icons are only rendered if ShowWidgetIcon setting is enabled.

Floating tab updates:

Render widget icon if ShowWidgetIcon setting is enabled.
Long labels are truncated.

I notice that if I set the tabs to "show icon only," the overflow shows the icon and the label. Is this intended?

I notice that if I set the tabs to "show icon only," the overflow shows the icon and the label. Is this intended?

Yeah, I believe that flag is only set by the overflow logic (after a certain size/breakpoint, hiding the label of a tab) allowing to display more tabs in a limited space w/o overflow. In case of an overflow menu there is enough space to show both the icon and the label.


Move ironic template defaulting to the webhook

Setting the values for Secret, DatabaseInstance and StorageClass in pkg/ironic makes the API result listing the spec for OpenstackControlplane not match actual settings used.
Also, defaulting in the pkg does not work as intended since the webhook call to ironic-operator explicitly set values such as the StorageClass for each conductor in "ironicConductors" - i.e setting StorageClass in the top level "ironics" in pkg/ironic never apply to the conductors. This caused invalid "" be defined for conductors when actual intent was to default it based on what was defined for top level ironics.
This change moves the defaulting of top level ironics properties to the webhook in openstack-controlplane, the defaulting happens before calling the Default() function in the ironic-operator.

/test precommit-check

All jobs look green. LGTM!


Cross-platform build and release process for single-file-executable

This works by making builds for windows in Appveyor python3.5_x64 and OSX builds in Travis using Py3.5.
Every PR build will go to a folder in S3 where it will be saved for 60 days.
Every time a tag is pushed, builds will be sent to a different folder in an S3 bucket where they can be collected and attached to the release.
Linux builds will still be made locally inside our special docker container, but that could also be automated with travis. It just doesn't seem necessary at this time.
We made special AWS credentials that just allow pushing to this one S3 bucket.
The DEV DOCS explain the process in more detail.

Coverage decreased (-0.03%) to 82.712% when pulling 55be3b884f3f53921711b6a5df9b34e10aa3da5f on matt/test_offline into e72ec671da44ca73cdbdafa9c2e5b8cbfda3d876 on master.

Coverage decreased (-0.03%) to 82.712% when pulling 55be3b884f3f53921711b6a5df9b34e10aa3da5f on matt/test_offline into e72ec671da44ca73cdbdafa9c2e5b8cbfda3d876 on master.

Coverage decreased (-0.03%) to 82.712% when pulling 55be3b884f3f53921711b6a5df9b34e10aa3da5f on matt/test_offline into e72ec671da44ca73cdbdafa9c2e5b8cbfda3d876 on master.

Coverage decreased (-0.03%) to 82.712% when pulling 55be3b884f3f53921711b6a5df9b34e10aa3da5f on matt/test_offline into e72ec671da44ca73cdbdafa9c2e5b8cbfda3d876 on master.

Coverage decreased (-0.03%) to 82.712% when pulling 55be3b884f3f53921711b6a5df9b34e10aa3da5f on matt/test_offline into e72ec671da44ca73cdbdafa9c2e5b8cbfda3d876 on master.

Coverage decreased (-0.03%) to 82.712% when pulling 55be3b884f3f53921711b6a5df9b34e10aa3da5f on matt/test_offline into e72ec671da44ca73cdbdafa9c2e5b8cbfda3d876 on master.

Coverage increased (+0.03%) to 82.768% when pulling 55be3b884f3f53921711b6a5df9b34e10aa3da5f on matt/test_offline into e72ec671da44ca73cdbdafa9c2e5b8cbfda3d876 on master.

Coverage increased (+0.03%) to 82.768% when pulling 55be3b884f3f53921711b6a5df9b34e10aa3da5f on matt/test_offline into e72ec671da44ca73cdbdafa9c2e5b8cbfda3d876 on master.

Coverage increased (+0.03%) to 82.768% when pulling 55be3b884f3f53921711b6a5df9b34e10aa3da5f on matt/test_offline into e72ec671da44ca73cdbdafa9c2e5b8cbfda3d876 on master.

Coverage increased (+0.03%) to 82.768% when pulling 55be3b884f3f53921711b6a5df9b34e10aa3da5f on matt/test_offline into e72ec671da44ca73cdbdafa9c2e5b8cbfda3d876 on master.

Coverage increased (+0.03%) to 82.768% when pulling 55be3b884f3f53921711b6a5df9b34e10aa3da5f on matt/test_offline into e72ec671da44ca73cdbdafa9c2e5b8cbfda3d876 on master.

The builds were uploaded successfully.
2017-02-23 12:35:30          0 osx-dev-builds-not-for-release/
2017-02-23 22:59:37   16675328 osx-dev-builds-not-for-release/datarobot_batch_scoring_55be3b884f3f53921711b6a5df9b34e10aa3da5f_executables.OSX.x86_64.tar
2017-02-23 17:16:38   16675840 osx-dev-builds-not-for-release/datarobot_batch_scoring_8b27202577dc96a9e99fce847aeb1d819a0c65be_executables.OSX.x86_64.tar
2017-02-23 20:46:12   16673280 osx-dev-builds-not-for-release/datarobot_batch_scoring_cbff7093ddebeb92b0de0e470788a654c68b398f_executables.OSX.x86_64.tar
2017-02-23 12:36:12          0 osx-tagged-artifacts/
2017-02-23 20:05:35   30379208 windows-dev-builds-not-for-release/datarobot_batch_scoring_55be3b884f3f53921711b6a5df9b34e10aa3da5f_executables.Windows.x86_64.zip
2017-02-23 14:43:07   30382348 windows-dev-builds-not-for-release/datarobot_batch_scoring_8b27202577dc96a9e99fce847aeb1d819a0c65be_executables.Windows.x86_64.zip
2017-02-23 18:16:29   30378889 windows-dev-builds-not-for-release/datarobot_batch_scoring_cbff7093ddebeb92b0de0e470788a654c68b398f_executables.Windows.x86_64.zip
2017-02-23 12:37:17          0 windows-dev-builts-not-for-release/
2017-02-23 12:38:00          0 windows-tagged-artifacts/

@datarobot/predictions Please review

Coverage increased (+0.03%) to 82.768% when pulling 5a80cf2574c764e6d4ac3c3ea1c2da5261bfead1 on matt/test_offline into e72ec671da44ca73cdbdafa9c2e5b8cbfda3d876 on master.

Coverage increased (+0.03%) to 82.768% when pulling 5a80cf2574c764e6d4ac3c3ea1c2da5261bfead1 on matt/test_offline into e72ec671da44ca73cdbdafa9c2e5b8cbfda3d876 on master.

Coverage increased (+0.03%) to 82.768% when pulling 5a80cf2574c764e6d4ac3c3ea1c2da5261bfead1 on matt/test_offline into e72ec671da44ca73cdbdafa9c2e5b8cbfda3d876 on master.

Coverage increased (+0.03%) to 82.768% when pulling 5a80cf2574c764e6d4ac3c3ea1c2da5261bfead1 on matt/test_offline into e72ec671da44ca73cdbdafa9c2e5b8cbfda3d876 on master.

Coverage increased (+0.03%) to 82.768% when pulling 5a80cf2574c764e6d4ac3c3ea1c2da5261bfead1 on matt/test_offline into e72ec671da44ca73cdbdafa9c2e5b8cbfda3d876 on master.


Rename enum to enumeration

This patch renames variables that have name enum to enumeration since
enum is a keyword for Dotty.

@gzm0 , I changed the commit message


ShortenedThrowableConverter: add option to HIDE elements from generated classes

It may come in handy to provide an option to automatically hide stack trace elements originating from generated classes.
For instance:
org.sample.SampleException: Unable to process request
	at org.sample.MyController.create(MyControllerController.java:41)
	at org.sample.MyController$$FastClassBySpringCGLIB$$340ae8ef.invoke(<generated>)
...

In this example, MyController$$FastClassBySpringCGLIB$$.. is a CGLIB class generated by Spring at runtime. It gives no additional value to report it in the stacktrace and users may want to remove/hide it to increase visibility.
The converter already allows to exclude elements matching a regex pattern. However a stack element is removed only if it is followed (or preceded) by at least one another excluded item. Also, the excluded items are replaced by a single line reporting the number of exclusions ("... 12 frames excluded").
Here we simply want to silently HIDE elements originating from generated classes so the exclusion is not applicable. This feature could leverage the existing StackElementFilter.withSourceInfo() filter and would be enabled using a simple flag as follows:
<filterGeneratedClasses>true</filterGeneratedClasses>

Note: how do we detect generated classes?
Is it enough to say that a class is generated if the filename and the line number are unknown? What if the code is compiled without debugging information?

@philsttr What's your opinion ?

I'd rather stick to the regexes.  There are going to be tons of edge cases for generated classes that I do not think logstash-logback-encode should try to detect.  We'll never get it right, and it sounds like a pain to maintain.
We could add another option to the ShortenedThrowableConverter to define the exclusion behavior.  Default being the current behavior.  New option for just marking exclusions with an asterisk or something.
something like this:
Original (non-shortened):
java.lang.RuntimeException: boom
    at com.foo.A.a()
    at cglib.foo()
    at cglib.bar()
    at com.foo.A.b()
    at java.lang.Thread.run()

Existing behavior (with a regex matching cglib.*)
java.lang.RuntimeException: boom
    at com.foo.A.a()
    ... 2 frames excluded
    at com.foo.A.b()
    at java.lang.Thread.run()

New option for marking where frames were excluded (with a regex matching cglib.*)
java.lang.RuntimeException: boom
    at com.foo.A.a()
   *at com.foo.A.b()
    at java.lang.Thread.run()


Release spark-history 2.4.0-2.2.1-3 (automated commit)

Release spark-history 2.4.0-2.2.1-3 (automated commit)
Description:
Source URL: https://s3-us-west-2.amazonaws.com/infinity-artifacts/permanent/spark-history/20181024-232739-V2RnOuejLbs4ngBq/stub-universe-spark-history.json
Changes between revisions 300 => 400:
0 files added: []
1 files removed: [resource.json]
2 files changed:
--- 300/config.json
+++ 400/config.json
@@ -7,7 +7,7 @@
       "properties": {
         "name": {
           "default": "spark-history",
-          "description": "The app name for the Spark History Server.  The service will be available at http://<dcos_url>/service/<name>/",
+          "description": "The app name for the Spark History Server.  The service will be available at https://<dcos_url>/service/<name>/",
           "type": "string"
         },
         "cpus": {
@@ -35,7 +35,7 @@
         "docker-image": {
           "description": "Docker image to run in.  See https://hub.docker.com/r/mesosphere/spark/tags/ for options.",
           "type": "string",
-          "default": "mesosphere/spark:2.3.1-2.2.1-2-hadoop-2.6"
+          "default": "mesosphere/spark:2.4.0-2.2.1-3-hadoop-2.6"
         },
         "hdfs-config-url": {
           "type": "string",
--- 300/package.json
+++ 400/package.json
@@ -20,6 +20,6 @@
     "analytics"
   ],
   "website": "https://docs.mesosphere.com/service-docs/spark/",
-  "version": "2.3.1-2.2.1-2",
+  "version": "2.4.0-2.2.1-3",
   "minDcosReleaseVersion": "1.9"
 }```

@imaxxs I just realized spark-history had not been released. The latest universe is still on 2.3.1-2.2.1-2. This update is for bumping to 2.4.0-2.2.1-3

@samvantran Is this PR going to be superceded shortly by @akirillov and @soumasish?
Shall we just wait once and for all for 2.5.0-2.2.1-3?

@vishnu2kmohan sure we can wait as long as you're okay with a missing spark-history-2.4.0-xx in the series


Incorrect expiry time in RefreshToken in OIDC?

Description:

My current setup is that token lifetime is 60 minutes and refresh token lifetime is 2 days (configured in identity provider). However, with the current expiry times envoy cookie model cannot just work. Lets say that I got new tokens 10:41:13Z, everything is stored to cookies. Normal token will expiry 11:41:13Z and after that it is supposed to use refresh token to fetch new normal token. However, that is impossible because refresh token cookie was expired 11:41:13Z as well and there are no cookie anymore. RefreshToken lifetime should be longer than normal tokens. Should it be configurable?
Repro steps:

Include sample requests, environment, etc. All data and inputs
required to reproduce the bug.

Note: If there are privacy concerns, sanitize the data prior to
sharing.

Environment:

Include the environment like gateway version, envoy version and so on.

Logs:

Include the access logs and the Envoy logs.

cc @zhaohuabing

@zetaab Can you confirm what's the expire_in field in the refresh token returned from the IDP?
If expire_in is not set, a default value will be abled to be applied to the refresh token TTL once #3423 and its implementation are merged.


Indonesia Call Center / Toll Free numbers

Hi, a few additional formats needed for Indonesia.  All tests done with v7.21:

Call center 5 digit numbers 140XX where X=0..9
Managed by Telkom http://www.infomedia.co.id/media/detail/id/58 (2nd paragraph, 2nd line)

Working Examples:
McDonald's 14045 http://www.mcdonalds.co.id/layanan/pesan-antar
KFC 14022 http://www.kfcku.com/home-delivery-14022
Bank Mandiri 14000 http://www.bankmandiri.co.id/article/faq-24jam.aspx
Note: If called from mobile we need to dial an area code before the 5 digits. Any area code works! Example: 021 14045 or 0274 14045
Current Status in Libphonenumber:
140XX isPossibleNumber() = true isValidNumber() = false
number with an area code is reported possible and valid
Need:
to add 140XX format as a valid format for Indonesia

Call Center 7 digit numbers 1500XXX where X=0..9 and its predecessor 500XXX
Managed by Infomedia Nusantara http://www.infomedia.co.id/media/detail/id/58 (2nd paragraf, last line)

Previously 6 digits (500 XXX) but the Ministry of Telecommunication changed that to add the 1 at the start in May 2014. 500XXX still can be used until 2016
http://www.postel.go.id/berita-migrasi-nomor-500xyz-menjadi-1500xyz-26-2192 (6th paragraf, 1st line)
Working Examples:
PHD 1 500 600 https://www.phd.co.id/hotpromo.aspx
Hokben 1 500 505 http://www.hokben.co.id/special_offer/detail/large-order-delivery-hokben
Bank BCA 1 500 888 http://www.bca.co.id/
Note: For 500XXX if called from mobile we need to dial an area code before the 5 digits. Any area code works! For 1500XXX no need for an area code.
Current Status in Libphonenumber:
500XXX and 1500XXX isPossibleNumber() = true isValidNumber() = false
Need:
to add 500XXX and 1500XXX format as a valid format for Indonesia

11 digit numbers (0804 XXXXXXX)
Managed by Telkom
Not official link https://hileudtaneuh.wordpress.com/2011/01/14/nomor-telepon-toll-free-dari-telkom/

Working Example:
Garuda Indonesia 0804 1 807 807 https://www.garuda-indonesia.com/id/id/contact/help.page?
Current Status in Libphonenumber:
0804 XXXXXXX isPossibleNumber() = true isValidNumber() = false
Need:
to add 0804 XXXXXXX format as a valid format for Indonesia

This will be in the next release.

@penmetsaa FYI

Hi Ben,
Thanks for reporting this issue.
LibPhoneNumber API do not support local phone numbers(The numbers without area code).
So we have taken the actions below based on your inputs and our investigation.

140XX: As this code goes to the same business with any area code when dialled from mobile, we are adding it to "Short Codes". For validation of short codes you need to use the below code.

PhoneNumber phoneNumber = PhoneNumberUtil.getInstance().parse(<your_StringInput>, <your_RegionCodeObject>);
ShortNumberInfo shortNumberInfo = ShortNumberInfo.getInstance();
shortNumberInfo.isValidShortNumber(phoneNumber)
When area code is present in the input, this range is already covered as valid landline number.

500XXX and 1500XXX: When area code is present in the input, these ranges are already covered as valid landline numbers. LibPhoneNumber API do not support local phone numbers. So not making any changes regarding 500XXX range.

Adding new range 1500XXX as valid UAN number. This range can be dialled without area code.

500XXXX: When area code is present in the input, this range is already covered as valid landline number.
As LibPhoneNumber API do not support local phone numbers, not making any changes on this regards.

804X XXXXXX: As per our analysis, adding this range to shared cost numbers.

Please let me know if you have any further questions. The new changes will be live in next release.
Thanks & Regards,
Aravind

Fixed in release 7.2.2


[tock] Make Tock's ePMP setup routine compatible with the Z1 ROM_EXT

Description
We should make the following changes to Tock's ePMP setup:

Remove use of entries 0 and 1, which the ROM_EXT locks. Use entries 2 and 3 to provide kernel execute permissions instead.
Disable entries 10-11 during startup (to remove executable permission for the ROM_EXT code).
Remove Tock's reconfiguration of entries 12-15, and instead lock them in place.

These changes affect upstream Tock code, and will therefore be dependent on upstream code review.

I discussed this with Leon, and we decided to introduce a new setup routine, rather than trying to have one setup routine that works with both a test ROM and ROM_EXT. This resulted in a much simpler setup routine.
PR at https://github.com/tock/tock/pull/4034


Improve docstrings

Can we improve our docstring practice to reduce duplication, improve clarity and generally set a standard?
I see this suggestion for python which seems to make sense to me: https://numpydoc.readthedocs.io/en/latest/example.html#example
In particular I like the lack of repetition of the function arguments, the use of doctests, and the use of a clear summary section and notes section for further details
Are there resources from the Julia community we could point at to justify/improve our practice?

https://github.com/JuliaDocs/DocStringExtensions.jl

Since we are using the SciML style for formatting; I assume we are going to use the SciML preferred documentation conventions?

what do you think? I think yes and its good to follow established standards and it seems like a high quality ecosystem

This makes me wonder if we do need this template builder for the doc strings or not?

Which template builder? DocStringExtensions?
I think its handy.

@seabbs @zsusswein
I've started putting this in on this branch; although far from finished version.
But have a look to see if that the level of doc you want.

Which template builder? DocStringExtensions?

Yes and to be clear i would like to  use it but not going to force the it on anyone

@SamuelBrand1 could you make make some sub issues for parts that still need docs so someone else can pick this up?

@SamuelBrand1 could you make make some sub issues for parts that still need docs so someone else can pick this up?

Yes, and not sure why I didn't just do that!

Basically, I'm happy on that branch until the bit documenting the callable of Renewal... thats clunky due to LaTeX strings (sure there is a better way).
Fairly random thoughts:

DocStringExtensions is probably most useful deployed asap, because it reduces the work load of redocumenting breaking changes.
I haven't used DocStringExtensions.@template but it looks handy? Again the kind of thing to set up before extensive documentation to save time.

I haven't used DocStringExtensions.@template but it looks handy? Again the kind of thing to set up before extensive documentation to save time.

Yes I think this is a good idea.

Lets break this down:

[ ] Finalize template, doc the EpiAware module and the abstract types and fall-back methods that dispatch on them as the most general case.
[ ] doc the utils
[ ] doc the epimodels
[ ] doc the latentmodels
[ ] doc the observationmodels
[ ] doc the inferencemethods

Something to note:

The @doc doc""" """ formulation from the Markdown standard library should be used whenever there is LaTeX.

This has been incrementally resolved by multiple PRs


SharePoint Adaptive Card - Dark Mode Color Issue

Target SharePoint environment
SharePoint Online
What SharePoint development model, framework, SDK or API is this about?
💥 SharePoint Framework
Developer environment
Windows
What browser(s) / client(s) have you tested

[X] 💥 Internet Explorer
[X] 💥 Microsoft Edge
[X] 💥 Google Chrome
[X] 💥 FireFox
[ ] 💥 Safari
[ ] mobile (iOS/iPadOS)
[ ] mobile (Android)
[ ] not applicable
[ ] other (enter in the "Additional environment details" area below)

Additional environment details
No response
Describe the bug / error
Now i have issue with ImageCard-HybridWork-Checkin Card when I change viva Sharepoint portal to dark mode the
the card below has some color issues which makes the input hard to see.
how to fix that

Steps to reproduce

Expected behavior
see black color with input

Any Update about this issue ?

I took a look at this and it looks like the form elements are not changing their text based on the theme of the SharePoint site. From what I could tell in the Adaptive Card documentation there isn't a color property on any of the form elements.

Hello All Thanks for support but is there any update with my issue ?

Hi @ahmedmahmoudR,
Thank you for bringing this to our attention. There was a service fix that was applied at the end of March for Dark Mode coloring. Would you mind checking to see if this still repos on your end as I'm not able to repo the issue without reverting back commits.

@micwilson911 I checked this in my environment and I can see a change. I used the Dark Yellow theme and the form fields look like this.

Looks like things are resolved although I didn't check all the themes.

dcashpeterson, thank you for checking. I'll close this issue out in the meantime.


RA parsing string incorrectly

   String p = "${@print}";
   given().get(p).then();

Gives me:
java.lang.IllegalArgumentException: Invalid number of path parameters. Expected 1, was 0. Undefined path parameters are: @print.
This comes from here:
com.jayway.restassured.internal.RequestSpecificationImpl.applyPathParamsAndSendRequest(RequestSpecificationImpl.groovy:1596)
Curl (or Postman) don't seem to mind this.

For these cases you need to manually perform URL encoding for your string before passing it to RA. RA treats everything inside curly braces to be parameters (and this is by design).

Hello,
I tried this, but this time the service return 404.


Replace Timeout methods with branching "run" methods

JAVA-5376
This removes all production usages of the methods. Remaining work pending feedback regarding approach. Please look for TODO-CSOT when reviewing, and treat these as PR comments - some are notes for work remaining to be done, some indicate places where the approach runs into issues, and some are questions/confirmations regarding CSOT.

Merged CSOT, 1 conflict: removed timeoutRemainingMS and its temp fix (see 4e97f21, "added temp fix for timeoutRemainingMS (scheduled to be removed)").


Failed to get access to local media ( Appr.tc)

Browsers and versions affected
Chromium V60.0.3112.89
OS : Debian Jessie
Hardware : Raspberry Pi 3
Description
Got error Failed to get access to local media ( Appr.tc) while entering chat room at https://appr.tc
Steps to reproduce
Open  https://appr.tc on chromium V60.0.3112.89
Expected results
Video call working without any error,
Actual results
Got error Failed to get access to local media , and can't stream local video

Please include javascript logs. What camera do you have?

I've got the same issue with USB camera Logitech HD Pro Webcam C920
Browsers and versions affected
Chromium V60.0.3112.89
OS : Debian Strecht
Hardware : Raspberry Pi 3
Description
Got error Failed to get access to local media ( Appr.tc) while entering chat room at https://appr.tc
Steps to reproduce
Open https://appr.tc on chromium V60.0.3112.89
Expected results
Video call working without any error,
Actual results
Got error Failed to get access to local media , and can't stream local video


MethylVI batch effect tutorial

Tutorial accompanies https://github.com/scverse/scvi-tools/pull/2834
Formatting

[x] My tutorial has only one top-level (#) header

Reproducibility

[x] My tutorial works on Google Colab
[x] My tutorial sets scvi.settings.seed = 0 at the beginning of the notebook
[x] My tutorial has been run and includes outputs (e.g. plots, tables)

Other

[x] Counts and normalized data should co-exist in the datasets, see the API overview for an example
[x] For scRNA-seq data, normalization should be counts per median library size and then log1p transformed -- if not, a reason should be given (Note: not applicable to this tutorial as we're not using scRNA-seq)

See comments overall looks fine. @ori-kron-wis can you help make the style consistent with other tutorials (I assume removing notebook style commands). These might change rendering on the webpage, which can end up ugly.

@canergen not so sure what is the problem/difference in style? care to give a few examples?

Hi @canergen. Just wanted to check in to see if you had updates re: my and @ori-kron-wis's previous comments.


lib: Bump version to 1.1.0

We'll support all APIs and features we have now for the forseeable future.
Why not 1.0.0? Just to avoid making this version feel too "special".

looks like CI does not like 1.1.0 :package:
error: failed to select a version for the requirement `bootc-lib = "^0.1"`
candidate versions found which didn't match: 1.1.0


Difference between GRS and GZRS

[Enter feedback here]
Can you please explain better on data in secondary regions for GRS and GZRS whether data in GRS will be copied asynchronously  to one physical server or to three physical servers ? Is the data in secondary region is what the main difference between GRS and GZRS ?
Thanks

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: f618f40d-ae53-5c3b-d8bf-0e80614f66d3
Version Independent ID: 93083407-8f1e-b3e2-6a62-e30be5e1fbcb
Content: Data redundancy - Azure Storage
Content Source: articles/storage/common/storage-redundancy.md
Service: storage
Sub-service: common
GitHub Login: @tamram
Microsoft Alias: tamram

@RevathiS23
Thank you for reaching out. At this time we are reviewing the ask and will provide an update as appropriate

@RevathiS23  Thanks for the update, I'd recommend posting your question in https://azure.microsoft.com/en-us/support/community/ since it's not directly related to the documentation. We'll follow up there. We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

@RevathiS23  Thanks for the update, I'd recommend posting your question in https://azure.microsoft.com/en-us/support/community/ since it's not directly related to the documentation. We'll follow up there. We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


I am unhappy to be forced to "your" timezone and AM/PM.

Timezone is now an argument (use --timezone="America/Los_Angeles" for the old behaviour.
The time format is RFC3999 (without microseconds) which is a good format for "the world". I think it is better to not push the AM/PM system to the rest of the world.

We can perhaps add timezone as an additional configuration in the refactored CLI #154, which will deprecate the current memgpt command.

@sarahwooders sounds good to me, as long one can simply use the current locale too. I was thinking about adding a parameter for am/pm vs 24h time, but I had problems with the application pick the corrent "last saved session". I think it used the wrong one because of string ordering issues when not using 24h time. But I did not dig into the code to verify that.

@cpacker I guess I did it wrong by not wearing flowers in my hair using the app ;-)

@oderwat we just merged the changes for the updated CLI which you can run with memgpt run. I'd recommend adding the timezone into the config https://github.com/cpacker/MemGPT/blob/main/memgpt/config.py#L39 and making it configurable in the CLI https://github.com/cpacker/MemGPT/blob/main/memgpt/cli/cli_config.py.

I will look at it in the next days (1 am here).

Well I have rebased the PR to current main and reformatted the code. But I actually have no idea on how to add the timezone to the new config setting in the correct way. So if @sarahwooders could help with that I would be very happy. I do think you need to let it run by default with the users timezone and use the RFC3999 format for creating the filenames, so they are sortable and meaningful for the user. One could also use UTC for them though.

@oderwat I'd recommend adding a timezone field to MemGPTConfig that is configurable to when you run memgpt configure (defined in memgpt/cli/cli_load.py). In, inside of the functions in utils.py, you can get the config with config = MemGPTConfig.load() to determine what timezone to use.
Let me know how that sounds / if anything is unclear!

You are still forcing that timezone on anybody.

Hey @oderwat I made a separate PR for this that just pulled local timezone by default, it's merged into main now: #298 (added you as a co-author on the commit).
The reason for this is I'm hesitant to add timezone as a command-line arg without properly adding it to the config setup we have right now (as @sarahwooders mentioned). Maybe that can come in a separate PR.

@cpacker I am happy with the local timezone, that is part of why I did not try harder with figuring out on how to add this to the config. But also, because I bet that @sarahwooders needs just some minutes to add it, while I still do not understand where it actually has to be placed. I am really not very much of a Python programmer. My last real production code was written with Python 2.x and WxWidgets back in 2009 I think :)


AWS SecretsManager implementation does not sync existing secrets to Kubernetes

What happened?

Crossplane's AWS SecretsManager implementation for Secret is a unidirectional implementation only. When trying to sync existing secrets from AWS SecretsManager to a Kubernetes Secret by using the external-name annotation, the expected Kubernetes secret is not being created. The other way around, creating an AWS SecretsManager secret from Crossplane works as expected.
How can we reproduce it?

Create a AWS SecretsManager secret
Try to reference that existing Secret with Crossplane's external-name annotation in your Secret spec
The secret in Kubernetes will not be created

What environment did it happen in?
Crossplane version: v1.1

AWS
EKS 1.19
Crossplane provider-aws 0.17.0

The issue you are describing could be solved by implementing LateInitialize in the secretsmanager.
However, if you are also referring to a constant bi-directional sync between AWS and K8s, I am not sure if this is possible. Because K8s secrets do - as far as I know - not expose something like a lastChangedTimeStamp it is not possible for the controller to decide which secret value is newer and therefore the desired one.
@muvaf  what do you think?

I was not thinking about a bi-directional sync. My usecase is to have a SecretsManager secret with automatic rotation and sync that to Kubernetes

OK, now I understand what you mean. Yes, secret rotation is currently not supported by Crossplane but could be implemented.
As a workaround for now, you could use external-secrets-controller to sync secrets from AWS to K8s.

I agree that auto rotation for secrets generated by Crossplane would be nice to have as well, but this what not my intention to be honest :-).
I'd like to store (e.g. manually or by some other automation) secrets in SecretsManager (i.e. to not store secrets in Git), then optionally add a rotation lambda function to it.
After that resource exists in AWS, I want to create a Crossplane Secret and have Crossplane reference the existing AWS resource and create a Kubernetes secret with the content of the secret.

So what you describe could be solved by external-secrets-controller, right?
What I could imagine is to add an option syncFromExternal to Crossplane's SecretsManager to sync a K8s to an AWS secret. But I'm not sure if this would follow the idea of Crossplane which aims to have K8s as a source of truth.
@negz any comments on that?

we will follow with #655  the late-init feature was implemented with #668


signup page will not redirect you to dashboard

If the contributor who is assigned this issue does not respond in 24 hours, I'll assign this to you.

Oky

Can i work on this issue as it is not assigned to anyone??


follower ids vs followers list

Wondering why when getting a list of followers (outside of the memory and slightly more complex logic) it uses the direct followers/list API endpoint rather than combindingfollowers/ids and users/lookup to return full user objects?
With User-Auth scope, you're looking at (per 15-minute window):

API
Max Return/Call Size
Requests / 15-min window
Total Results Per Window

followers/list
200
15
3000

followers/ids
5000
15
75000

users/lookup
100
180
18000

So getting the IDs then looking up via users/lookup will get you the same thing as followers/list but 6 times quicker for larger result sets, unless I'm missing something?

This isn't a tweepy question - might ask on a twitter API forum. (Pure speculation: maybe the lower results reflects that the former's cursoring etc is a more significant system load.)

Well it kinda is, since tweepy by default uses followers/list for everything, even though it could (and should) probably use the latter method if the requested count exceeds a certain threshold.

Best to leave that to the user code, as "users/lookup" might already be a bottleneck, and having the API add load to it under the covers would be annoying in such a case.

I'm curious where you got the max return values from: have they been removed or am I missing something?

Again, this is not a tweepy issue. See the twitter API docs for things like "max return" for a call. This tweepy ticket should be closed.


Proposal for automated checks of knowledge submission

In order to release knowledge contributions to open source community, IBM legal is asking us to propose an automated solution for our generation pipeline that covers:
On the provided knowledge source from the contributor:

HAP detection
PII detection
Screening for commonly copyrighted content (e.g. OneShield) -- lower priority

On the synthetic data created from the knowledge source

Screening for leakage of text from the knowledge source being used verbatim in the synthetic data (leveraging OneShield)

Closing as we now work with Red Hat legal.


Create README.dev.md

Description

Related issues:

#24

Instructions to review the pull request
Review online.

Thanks for the review. I will add this then and maybe work on improving.


Is endpointElement really required

          LGTM

We have to consider if EndpointElem is really required in the code, given that we already have VPCResourceIntf. This may allow us to reduce code duplication, as Node also embeds VPCResourceIntf. This can wait for now, but maybe open an issue for that...
Originally posted by @zivnevo in https://github.com/np-guard/vpc-network-config-analyzer/pull/136#pullrequestreview-1580452366

PR #222 has replaced  EndpointElem with VPCResourceIntf on connectivity structures before grouping.
The  EndpointElem type is currently used to represent grouped elements.

The delta between EndpointElem  and VPCResourceIntf  is that the latter interface implements only Name() while the latter also implements UID(), Name(), ZoneName() and Kind(); the former functionality is non-relevant for grouped elements but required for VPC resources. Thus, the answer seems "yes",  EndpointElem is required


Add test fixture for k8s docker cluster

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#1185  👈
#1180
#1174
#1173
#1172
#1171
#1170
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @carolineechen and the rest of your teammates on  Graphite

Merge activity

Aug 29, 5:33 PM EDT: @carolineechen started a stack merge that includes this pull request via Graphite.


Analyzer shows a white page for a simple profile

Description
Analyzer shows a white page for a simple profile
Observed behavior
Analyzer shows a white page for a simple profile
Expected behavior
Show a list of suggestions.
Step-by-step guide on how to reproduce the bug
Upload a profile.
Additional context
...
Environment

Browser: Chrome
Operating System: Linux
Other:

Output
main.js:8 Error: Could not find item with key 99d73fec-6d20-4322-aacc-02d3c2b31cfd in local storage.
    at Jwt (main.js:22:176696)
    at main.js:37:164836
    at Rx (main.js:8:24501)
    at Wp (main.js:8:43140)
    at main.js:8:41441
    at iU (main.js:1:30857)
    at MessagePort.tU (main.js:1:31257)
main.js:22 Uncaught Error: Could not find item with key 99d73fec-6d20-4322-aacc-02d3c2b31cfd in local storage.
    at Jwt (main.js:22:176696)
    at main.js:37:164836
    at Rx (main.js:8:24501)
    at Wp (main.js:8:43140)
    at main.js:8:41441
    at iU (main.js:1:30857)
    at MessagePort.tU (main.js:1:31257)

File(s) scanned
Optionally provide file(s) that demonstrate the bug when running the tool.
Ideally, do not attach files directly. Bazel profiles can include sensitive data, such as exposing
which dependencies your project includes.
Instead, upload files to the cloud storage provider of your choice and grant appropriate access.
This allows you to adjust who can access them now and in the future. For example, you may want to
remove access to the files once the issue has been resolved.
Further details
Provide other data related to this issue that may prove helpful.

@saraadams

I found two issues and have fixes for them.
White page: this is caused by an uncaught error.
Cause of white page: This looks to be a race condition in play, which is why reloading the "white page" may show the results.
The fixes will be part of the next release.
If you encounter this bug before the new release has been rolled out, try refreshing the white page whenever you encounter it on analyzing a profile. It looks like the race condition is at least much rarer (or plays out in our favor) when you reload the page.

This is now rolled out and was confirmed as fixed by the initial reporter.


Added dependency versioning for System.IdentityModel.Tokens.Jwt

We are always getting complaints that we should not allow updates to System.IdentityModel.Tokens.Jwt - this patch provides appropriate dependency checking.  See https://docs.microsoft.com/en-us/nuget/create-packages/dependency-versions for info on dependency checking.

Fixes #195


enhancement on issue #196

To specifying the demand of issue #196. I did research on types of whitespace and I pick some most common whitespace: space, tab, line feed, form feed, carriagereturn. Then I modified equalToCompressingWhiteSpace by adding a property whiteSpaceType which is a enum type variable with 6 types of whitespace: space, tab, line feed, form feed, carriagereturn, mix(mix types above). From this, I modify the constructor. To change the match rule, I modified stripSpaces to strip string according to different types of whitespace. To prevent conflicts, I added 6 types of matcher generators,  public static Matcher<String> equalToCompressingSPACE(String expectedString) as an example.

Going to try and kick start hamcrest, so if you want to get it merged, please rebase from the branch v2.3-candidates.
Still trying to understand how has permissions to perform a release.


Support host sharing and guest pulling for Confidential Container image management

Support host sharing and guest pulling for Confidential Container image management:

import a KataVirtualVolume structure, which acts as a superset of ExtraOptions
support build a disk image for a container image with EROFS and passing disk image info to kata-runtime through KataVirtualVolume.
support downloading container image inside guest by introducing proxy mode, which doesn't download any layer content on host but only insert a KataVirutalVolume into Mount.Options.

This PR is based on #528

May be add some usage docs later.

Maybe also need to take some E2E tests?

Maybe need to add some unit tests, coverage has dropped too much.

Due to the tight schedule, we will add doc/tests with following-on PRs.


go-import meta tag is wrong

Gogs 0.11.29 (or commit ref):
Git version:
Operating system: Linux
Database (use [x]):

[x] PostgreSQL
[ ] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gogs.io:

[ ] Yes (provide example URL)
[ ] No
[ ] Not relevant

refs #1878 ,
when trying to go get my.org/me/repo/subdir, it clones whole my.org/me/repo.git into the subdir directory, I expect only contents of subdir to be there.
reason:
go get expects a meta tag, that in it's content we have following:  "GOPATH_install_dir VCS VCS_Address".
I checked on github, and for example meta tag for cockroachdb sub-pkgs are:
request: curl https://github.com/cockroachdb/cockroach/tree/master/pkg/sql?go-get=1
response: meta name="go-import" content="github.com/cockroachdb/cockroach git https://github.com/cockroachdb/cockroach.git"
while gogs 0.11.29 generates:
meta name="go-import" content="my.org/me/repo/subdir git https://my.org/me/repo.git"
the subdir at the end of "my.org/me/repo/subdir" is extraneous and leads to incorrect behavior.
please fix.

I have the same issue.
To sum-up: the meta go-import should always point to the root of the repo, not to the subdir requested.

There is a fix in #4726.

glad to see this is being fixed - we've had a couple gogs users show up in the dep queue with issues arising from this problem: golang/dep#1213 golang/dep#1120.
the bug makes it impossible to use dep (and likely at least difficult for other tools) to reference non-root packages in repositories stored with gogs. the fix can't land soon enough 😄
also, just to reiterate the OP's information by quoting from the official go toolchain docs:

If the import path is not a known code hosting site and also lacks a version control qualifier, the go tool attempts to fetch the import over https/http and looks for a  tag in the document's HTML .
The meta tag has the form:
<meta name="go-import" content="import-prefix vcs repo-root">

The import-prefix is the import path corresponding to the repository root. It must be a prefix or an exact match of the package being fetched with "go get". If it's not an exact match, another http request is made at the prefix to verify the  tags match.

Already fixed in develop branch, thanks everyone!


consensus: add Receipts struct

Related https://github.com/paradigmxyz/reth/issues/7238

@mattsse Nice bump here, can we merge or any blocker?

sorry, forgot about this, one nit

@mattsse done


delete zju.edu.cn from stoplist

Zhejiang University is a top school in the field of computer science and its offical email is zju.edu.cn. I think There must have been some mistake that made Zhejiang University on the list.

@apache1011 No, this is not a mistake. We found out that someone sells accounts with this email domain on TaoBao.
It means that someone from the college shared their account details publicly, or sold it, and it led to broad account misuse and license terms violation.
Therefore, we are not able to identify if the people with @zju.edu.cn  addresses are actual students or not.
If you are the student of this college, please apply with official document ("OFFICIAL DOCUMENT" tab at https://www.jetbrains.com/shop/eform/students)


[Question] Very simple setup example

Hi there,
unfortunately I could not get it to work. Can you point out, what I might be doing wrong? I guess this is a very typical starting point, so it might be interesting for other users too.
My Vertex Shader:
cbuffer cbPerFrame : register(b0)
{
	float4x4 World;
	float4x4 Projection;
	float4x4 View;
};

struct VSInput
{
	float3 Position : POSITION;
	float3 Normal : NORMAL;
};

struct VSOutput
{
	float4 Position : SV_POSITION;
	float4 Color : COLOR;
};

VSOutput main(VSInput vin)
{
	VSOutput vout = (VSOutput)0;

        float4 worldPosition = mul( float4( vin.Position, 1.0f ), World );
        float4 viewPosition = mul( worldPosition, View );
        vout.Position = mul( viewPosition, Projection );
	vout.Color = 1;

	return vout;
}

And the Pixel Shader:
struct PSInput
{
	float4 Position : SV_POSITION;
	float4 Color : COLOR;
};

float4 main(PSInput pin) : SV_TARGET
{
	return pin.Color;
}

I set World to MatrixIdentity, View to MatrixLookAtLH with Eye: (10,5,10), Focus: (0,0,0) and Up: (0,1,0), and Projection to MatrixPerspectiveLH with View size: (800,600), Near: 0,1 and Far: 100.
But I can see nothing. Any ideas?
Cheers,
Georg

Hey there,
Just tested your example and it should work. Did you name your variables in the "Variable Manager" window exactly "World", "View" and "Projection"? SHADERed requires that names in that window must match the names in the shader code.
Here's my project:
HLSLsimple.zip
You can also assign a "system value" to your matrix. These will update parameters automatically (for example: viewport size)
There are also plenty of examples in the examples directory.
You can also check this small tutorial I've written: https://github.com/dfranx/SHADERed/blob/master/TUTORIAL.md

Thank you. I'll try that.


Blank Page after install

Hey, i installed it following the https://docs.cachethq.io/docs/installing-cachet instructions
and i get a blank white page but the favicon showing.
Am running CentOS 7.1
PHP Version: 5
Apache Version: 2.4.6
MySQL Version: 5.5.44-MariaDB

I'm having the same issue.

PHP Version: 5

Can you be more specific please?

Are the file permissions wrong on your storage folder/

PHP 5.5.28 (cli) (built: Aug  9 2015 12:22:11)
Copyright (c) 1997-2015 The PHP Group
Zend Engine v2.5.0, Copyright (c) 1998-2015 Zend Technologies

Your storage folder must belong to the wrong user then.

The user running php must have write access to the storage folder.

If you like, you can just recursively set the permissions to 755 or something, but I'd consider that to be insecure given that code injected into the storage folder can endup being executed by php (compiled views).

i tryd doing "chmod 755 -R storage" still blank page

"root:root" got access to everything in the folder

I'm getting same issue and I'm using nginx.

I can't do anything unless you actually give me logs.

In Apache logs its showing:
[Sat Aug 29 08:18:21.103380 2015] [:error] [pid 19762] [client ] PHP Fatal error:  Uncaught exception 'UnexpectedValueException' with message 'The stream or file "/var/www/Cachet/storage/logs/laravel-2015-08-29.log" could not be opened: failed to open stream: Permission denied' in /var/www/Cachet/bootstrap/cache/compiled.php:12857\nStack trace:\n#0 /var/www/Cachet/bootstrap/cache/compiled.php(12945): Monolog\Handler\StreamHandler->write(Array)\n#1 /var/www/Cachet/bootstrap/cache/compiled.php(12795): Monolog\Handler\RotatingFileHandler->write(Array)\n#2 /var/www/Cachet/bootstrap/cache/compiled.php(12558): Monolog\Handler\AbstractProcessingHandler->handle(Array)\n#3 /var/www/Cachet/bootstrap/cache/compiled.php(12662): Monolog\Logger->addRecord(500, Object(Symfony\Component\Debug\Exception\FatalErrorException), Array)\n#4 /var/www/Cachet/vendor/graham-campbell/exceptions/src/ExceptionHandler.php(79): Monolog\Logger->critical(Object(Symfony\Component\Debug\Exception\FatalErrorException), Array)\n#5 /var/www/Cachet/bootstrap/cache/compiled.php(1850): GrahamCampbell\Exceptions\ExceptionHandler->report(O in /var/www/Cachet/bootstrap/cache/compiled.php on line 12857
Any suggestion/fixes?

Yeh, you're file permissions are wrong.

by doing "chown -R apache:apache /var/www/Cachet" i got it fixed :) thanks for the help :+1:


Full web API access in LiveView with WSDOM

I made WSDOM, which is a Rust→JavaScript RMI. It's like web-sys, but runs over WebSocket instead of in WebAssembly. If integrated into Dioxus, it would allow Dioxus LiveView apps to access WebGL, Canvas, and all the other Web APIs.
I think Dioxus, as one of the few Rust frameworks with LiveView feature, might be interested. If you want to add WSDOM to Dioxus, I would be happy to help!
(sorry if this issue comes of as an ad for my library; feel free to close or delete)

Hey! Yes, we'd love to play around with this - it seems awesome. If you want, we're very active in discord and would be happy to chat through integration. We also have office hours on April 3 if you want to chat live.


Proper way to modify ScrollView top or contentInset

Hey guys, my problem is that I have a ScrollView with a couple of text fields, and i want to for the view to scroll up whenever a field is focused (or keyboard) opens. At the moment I have
<ScrollView contentInset={{top: (this.state.keyboardOpen) ? -100: 0}}>
   <TextInput onFocus={() => this.setState({keyboardOpen: true})} />
</ScrollView>

which changes the state correctly but the scrollview doesn't scroll, if i manually scroll just a bit then the view updates and uses the new contentInset but it doesn't happen automatically.
Is there a better way to do this with RN?

This is kind of tricky to do with React's programming model but I guess I would use an onScroll handler to track the current contentOffset. When the keyboard opens, increase the bottom contentInset and then call scrollTo(trackedContentOffset + keyboardHeight).

@ide how do you call scrollTo() on a ScrollView?

Something like this.refs.scrollView.scrollTo. The React documentation talks about refs and is a good place to learn if this is the first time you're using them.

@ide ah of course! Just not used to it yet in RN....still in the "holycrap this all really works" stage lol thanks i'll give that a try.

@ide Happy to say this worked perfectly. Thank you.

@josebalius Can you post your solution?

@dimoreira sure here is what I did
<ScrollView ref="scrollView">
   <TextInput onFocus={() => this.refs.scrollView.scrollTo(100)} onBlur={() => this.refs.scrollView.scrollTo(0)} />
</ScrollView>

Hope this helps

@ide do you know if any changes have gone into the scrollview animation? This same functionality that I implemented now scrolls really slow, before it was fluid but after updating to 0.4.4 it is pretty slow (almost 2 seconds when it used to take 0.5 at most)

@josebalius: I'm not aware of what could have caused this slow down. Doesn't seem good though - can you open a new issue with more detail?

@ide nvm :( it was this: http://stackoverflow.com/questions/8624519/simulator-slow-motion-animations-are-now-on somehow I had toggled that on....sorry about that.


[AutoRelease] t2-azurestackhci-2024-08-16-34992(can only be merged by SDK owner)

https://github.com/Azure/sdk-release-request/issues/5372
Recording test success
https://dev.azure.com/azure-sdk/internal/_build?definitionId=1851

issue link:https://github.com/Azure/sdk-release-request/issues/5372

/check-enforcer override


Fix first doc line being ignored

Maintainer merge checklist

[ ] Title is descriptive/clear for inclusion in release notes.
[ ] Applied a Component: xxx label.
[ ] Applied the api-deprecation or api-break label.
[ ] Applied the release-highlight label to be highlighted in release notes.
[ ] Added to the milestone version it was merged into.
[ ] Unittests are included in PR.
[ ] Properly documented, including versionadded, versionchanged as needed.

Somehow the first line is still being ignored: https://kivy.org/doc/master/api-kivy.graphics.html#kivy.graphics.InstructionGroup

Sadly this didn't fix the issue.


Kafka 4738:Remove generic type of class ClientState

Remove generic type of class ClientState. Also removed Generic type from TaskAssignor as it referred to same generic type of ClientState

@sharad-develop Please review the PR changes and list of commits before submitting them. This is the 4th PR we have sitting open that has a ton of unrelated commits (I assume because you are working off a branch like 0.10.2 and then submitting PRs against trunk).

Retest this please

@sharad-develop Can you please fix the rebasing issue? There are 4 PRs with the exact same issue and none of them will be merged (and likely nobody will review them) until this is fixed.


Move from Oracle JDK to (adopt) OpenJDK

Proposal for Java support in Egeria (1).docx
Originally proposed in EGERIA-2 & moving here.
This was originally opened to track changing our recommended JDK from oracle to OpenJDK due to changes in licensing with the Oracle JDKs. The attached document provides more information
As of 24 Sep openJDK on mac, windows will work for building Egeria, but we noted the following issues
https://github.com/AdoptOpenJDK/openjdk-build/issues/387 - Originally we were checking exact JDK versions in the Egeria build, but the adoptopenjdk builds had a broken version string. The fix at the time was to only check for major version/release ie '1.8' and to reintroduce at a later date. I now suggest we stick with major version/release only.
https://github.com/AdoptOpenJDK/openjdk8-releases/issues/11 - the Windows openjdk release was missing cacerts files, which would cause pulls of new components from maven central to fail - this change was completed today
This issue will track any remaining changes ie to readme/documentation/mailing list to 'complete' our switched position on recommended JDK.

Note also for ubuntu 18.04 https://bugs.launchpad.net/ubuntu/+source/ca-certificates-java/+bug/1770553

I reported an issue with AdoptOpenJDK openJ9 not working correctly with IntelliJ.
The hotspot JVM works ok
This has been investigated by the OpenJ9 team, and has resulted in a request against the
IntelliJ team to support J9
https://github.com/eclipse/openj9/issues/3441
Still awaiting hotspot builds that address both issues cacerts, versions above cross platform


Remove mvc dependency

hi,
This issue fixes #8
I worked on the remove_mvc_depency branch.
I basically refactored the code into Swagger.Core and creted Swager.WebApi with specifics for the WebApi project.
I left the original Swagger project with files that are only specific to MVC.

@allansson and @domaindrivendev  I'm interested in having swagger-ui isolated from the API itself (hosting), and I don't mind referencing the binaries to use it, how can I achieve that using Swashbuckle?
If you could lend me some help I'd appreciate it.


Replace "editor.renderWhitespace" with independent settings for leading, trailing, inner, only, selection

The current setting "editor.renderWhitespace" can have 5 values - "none", "all", "selection", "trailing", "boundary". This doesn't cover some reasonable use cases. Adding a pile of other value strings to enable these cases would be unwieldy.
I suggest instead using 5 separate variables for:

Leading whitespace (for non-whitespace-only lines)
Trailing whitespace (for non-whitespace-only lines)
Inner whitespace (whitespace between non-whitespace characters)
Only whitespace (whitespace-only lines, important for the primary use case and prevents ambiguity)
Selection whitespace (overrides other values)

These values could be simple booleans, but for added customizability they could take values "none", "spaces", "tabs", "all".
The main use case here is to avoid showing tabs on whitespace-only indented lines. However, adding full user customization would solve all future issues of this type.
(Related to https://github.com/microsoft/vscode/issues/43127#issuecomment-499489078, but this is about auto-trimming)

Note, the main duplicate (https://github.com/microsoft/vscode/issues/54210) was closed due to similarity with another thread that was marked solved but only affected auto-trimming.


copyit line-endings are causing false fixity failure reports

As reported by @b00giehead:
copyit will sometimes say that the manifests were not identical, even though the only difference is line endings.
This change in line endings happened around the time of the python3 fixes, where I stopped opening text files in binary mode. Opening textfiles in binary mode on python2 was an easy way to make sure that Windows and Unix would use the same line endings, but that has all changed now.
Joanna and I have both noticed that a manifest made in windows can trigger this warning if copyit is used on OSX, and maybe vice-versa.
I am pretty sure that I've just encountered an issue where a manifest made on Windows prior to this change has now triggered the issue after a copyit transfer.
The solution is most likely to just ignore newlines when comparing manifests. This should be simple - I THINK????

https://github.com/kieranjol/IFIscripts/blob/master/copyit.py#L573 this is the line in particular.
So this page might have an answer but heavy testing is needed: https://stackoverflow.com/questions/42981652/how-to-ignore-different-newline-char-with-python-filecmp

I added a much more general method of comparison that ignores line endings, thanks for reporting this @b00giehead

Thanks Kieran. I’ll try this when back in office next week :)


Error trying to create a chart based on a dataset

I have a working query with a set a columns. When I try to save it and create a chart I get the following error in the UI:
Unexpected error: Columns missing in dataset
I tried to add / remove columns and it did not help. There are no errors in the log. I'm using the latest superset docker image

Hello, this would be a good candidate for posting in the Superset Slack chat as it's more of a request for troubleshooting help than a bug report.  It's unlikely though possible this is a bug in Superset; more info would be needed to determine either way.  One way you could test would be to replicate the problem on a stable Superset release (e.g., 3.0.2).  If the same thing doesn't throw an error there, it would indicate a possible bug on the the latest image.
Are you using Jinja templating?  If so, you may need to remove the Jinja temporarily to save the dataset or refresh columns.

I'm using the docker image, and from looking at the tags I did not understand how do I get the latest stable image - so I used the default. Maybe that's the source of the problem - if you could write the name of the tag I should use it would be great.
I'm not using Jinja templates. I did reinstall superset and recreate the query and dataset before I opened this issue - I also added and removed columns but it did not help.

if you could write the name of the tag I should use it would be great

Hi @ori8954. You can follow the Quickstart guide to obtain the latest stable image. The latest stable version is 3.0.2.

I would expect to get the latest stable docker image where no version is specified, I'm sure it can save trouble for the docker users.
I found the issue - I'm using SQL Server as a database and I had the reserved keyword 'data' in my query. While it runs as expected in the SQL Lab it fails to run after saving it as a dataset.
You can try the following query to reproduce the error:
WITH data AS (
SELECT 1 AS column_name)
SELECT *
FROM data
If I change the name 'data' to another name problem is solved. This query gives an SQL error + missing columns error. The  original query I had did not give a SQL error

Thanks for the additional info, good troubleshooting.  If you don't mind I'm going to change the issue title to be more specific.
I also use SQL Server.  I agree I can run the query you shared in SQL Lab successfully (Superset 3.0.0 for me).  When I save as a dataset, I get a different error message on the chart building view:

Error: (102, b"Incorrect syntax near ','.DB-Lib error message 20018, severity 15:\nGeneral SQL Server error: Check messages from the SQL Server\nDB-Lib error message 20018, severity 15:\nGeneral SQL Server error: Check messages from the SQL Server\n")

I clicked "View Query" to see what Superset is sending to SQL Server and it's this:
WITH data,
     __cte AS ( AS (SELECT 1 AS column_name)SELECT * FROM data)
SELECT TOP 1000 column_name AS column_name
FROM __cte;

Which errors when I run it directly in SQL Server too.
When I repeat the test but with data2 instead of data, it works.  This is the SQL generated and sent to SQL Server:
WITH data2 AS
  (SELECT 1 AS column_name),
     __cte AS
  (SELECT *
   FROM data2)
SELECT TOP 1000 column_name AS column_name
FROM __cte;

Different enough that maybe the problem is before SQL Server, it's with how the query is getting shaped along the way?  I wonder if others can replicate this with a different backend.

Docker builds and docs have been significantly updated, and this thread has been quiet for a long time. Also, it's a support request more than a bug. Closing due to all of those reasons, but please do file a new issue with fresh context if you're still experiencing docker issues... we're trying to make it easier.


[BUG] Invoke-PnPSPRestMethod - Pages API - Publish - The input does not contain any JSON tokens

Reporting an Issue
Using the Pages api through SharePoint REST API v2, this command:
Invoke-PnPSPRestMethod -Url "https://<tenant>.sharepoint.com/_api/v2.0/sites('<tenant>.sharepoint.com,00000000-0000-0000-0000-000000000000,00000000-0000-0000-0000-000000000000')/pages/00000000-0000-0000-0000-000000000000/oneDrive.publish" -Method "POST" -ContentType "application/json" -Content @{} -ErrorAction Stop

(all guids replaced with empty guid)
returns:
The input does not contain any JSON tokens. Expected the input to start with a valid JSON token, when isFinalBlock is true. Path: $ | LineNumber: 0 | BytePositionInLine: 0.

Expected behavior
Publish the Modern Page
Actual behavior
Error message as depicted above.
NOTE1: it does not matter if the 'Content' parameter is omitted
NOTE2: it does not matter if the Url suffix 'oneDrive.publish' is replaced with 'publish'
NOTE2: the exact same call passes without error through a direct Graph call using EasyGraph Module and Invoke-PnPGraphMethod:
Invoke-PnPGraphMethod -Method POST -Url "beta/sites/<tenant>.sharepoint.com,00000000-0000-0000-0000-000000000000,00000000-0000-0000-0000-000000000000/pages/00000000-0000-0000-0000-000000000000/publish" -ContentType "application/json" -ErrorAction Stop

What is the version of the Cmdlet module you are running?
1.12.0
Which operating system/environment are you running PnP PowerShell on?

[x] Windows
[x] Linux
[ ] MacOS
[ ] Azure Cloud Shell
[ ] Azure Functions
[ ] Other : please specify

Thanks @SjoerdV for raising the issue.
Can you please tell us how to get actual GUID values ? Will try to reproduce and fix this :)

Thanks @SjoerdV for raising the issue.
Can you please tell us how to get actual GUID values ? Will try to reproduce and fix this :)

Sure @gautamdsheth
make these GET calls to the following endpoints
/sites?select=id,webUrl,displayName # resulting id = id of the site
/sites/$siteid/pages?select=id,title # use the 'id' of a site in the previous call, resulting id = id of the page

Thanks @SjoerdV for raising this issue, have fixed it.
Was an interesting edge case we missed.
The fixed version will be available in tomorrow's nightly build and the next major version.

Thanks for solving @gautamdsheth !

confirmed working with nightly 2.0.12 on pwsh 7.3.2.