change the signature of methods to accept class object

@cyb3rko hi ... can we do this in kotlin? I have changed the signature of call method to accept HelloworldCallRequest instead of String

package com.m3o.m3okotlin.services

import com.m3o.m3okotlin.M3O.getUrl
import com.m3o.m3okotlin.M3O.ktorHttpClient
import com.m3o.m3okotlin.WebSocket

import io.ktor.client.request.*
import kotlinx.serialization.decodeFromString
import kotlinx.serialization.encodeToString
import kotlinx.serialization.json.Json
import kotlinx.serialization.Serializable

private const val SERVICE = "helloworld"

object HelloworldService {
    suspend fun call(req: HelloworldCallRequest): HelloworldCallResponse {
        return ktorHttpClient.post(getUrl(SERVICE, "Call")) {
          body = req
        }
    }
    fun stream(req: HelloworldStreamRequest, action: (Exception?, HelloworldStreamResponse?) -> Unit) {
        val url = getUrl(SERVICE, "Stream", true)
        WebSocket(url, Json.encodeToString(req)) { e, response ->
            action(e, if (response != null) Json.decodeFromString(response) else null)
        }.connect()
    }
}
@Serializable
internal data class HelloworldCallRequest(val name: String)
@Serializable
data class HelloworldCallResponse(val message: String)
@Serializable
internal data class HelloworldStreamRequest(val messages: Long, val name: String)
@Serializable
data class HelloworldStreamResponse(val message: String)

and then use it like this
package examples.helloworld.call

import com.m3o.m3okotlin.M3O
import com.m3o.m3okotlin.services.HelloWorldService

suspend fun main() {
    M3O.initialize("M3O_API_TOKEN")

    req = HelloWorldServiceRequest(name = "Jone")

    try {
        val response = HelloWorldService.call(req)
        println(response)
    } catch (e: Exception) {
        println(e)
    }
}

That's an idea, it should work.
You are only missing the imports and the val in the main method when initializing the req.

And how do you implement e.g. the stream request to follow the code scheme we use here?
Because I have some improvement, where can I at some point contribute it?

Because I have some improvement, where can I at some point contribute it?

You can contribute here (this org is basically a lab) until you PR kotlin project to m3o/m3o and m3o/m3o-kotlin.

And how do you implement e.g. the stream request to follow the code scheme we use here?

package examples.helloworld.stream

import com.cyb3rko.m3okotlin.M3O
import com.cyb3rko.m3okotlin.services.HelloWorldService

fun main() {
    M3O.initialize("NjJhODlhMmMtOGQ3Ni00YzljLWJhMzAtNGFkYzBhZTc5Yzcz")

    val req = HelloworldStreamRequest(messages = 2, name = "John")

    try {
        val socket = HelloWorldService.stream(req) { socketError, response ->
            if (socketError == null) {
                println(response)
            } else {
                println(socketError)
            }
        }
    } catch (e: Exception) {
        println(e)
    }
}

I think the solution I use in my handcrafted client is more convenient, but if Asim says he wants it like that, I can build new examples for that.

@cyb3rko Hi ... back to this .... i have applied the changes that we have discussed above ... can you make some random tests of these newly generated clients, lets see if they work properly.

Hi, sorry for the delay. I was on holidays after finishing school.
I will take a look at it today.

Congrats... and don't worry

Okay.
Some other things we have to change:

Every data class has to be public now (so the Request data classes are not allowed anymore to be "internal", just remove the keyword).

The first line of every file currently is "package com.m3o.m3okotlin.services".
But it has to end with the according name of the service. E.g. "package com.m3o.m3okotlin.services.helloworld".

Furthermore we just need to fix a broken import line, I can open a pull request.

Cool... I will fix the generator... Then we can work on generating examples for each endpoint based on the examples.json

One thing that we can do to improve the visibility (it is quite verbose right now) is to create an index file where we list all repetitive imports in one file and then use that file instead, so for example at the top code there are 7 repeated import statements that we could replace with only one import.

If I understand right you want to remove the following lines of the example above:
import com.m3o.m3okotlin.M3O.getUrl
import com.m3o.m3okotlin.M3O.ktorHttpClient
import com.m3o.m3okotlin.WebSocket

import io.ktor.client.request.*
import kotlinx.serialization.decodeFromString
import kotlinx.serialization.encodeToString
import kotlinx.serialization.json.Json
import kotlinx.serialization.Serializable

Unfortunately that's not a thing you can do in Kotlin.
And honestly I don't see a reason why we should do changes about the imports. Additionally they are just necessary for the code to work xD
We could just lower the number of imports with wildcarding.
You can even see this in code of Google Apps like here:
https://github.com/material-components/material-components-android/blob/master/catalog/java/io/material/catalog/main/MainActivity.java#L21-L38
Even though you could use more wildcard imports here for imports but they don't even use them here.
What we can improve though is to use better formatting:

remove blank lines between imports
sorting the imports alphabetically (here they seem to be sorted, I don't know about the other files)

I see... One more thing that we need to know for sure is the process of publishing a library to the kotlin libraries/packages manager... What are the steps we need to do? Is there any community github action that do all/parts of the process?

For sure. I will look it up and tell you when I've found something.

But first let's make sure one single line of code is executable and does not end in 5 errors xD
And mabye I should then try to test more then only HelloWorld.

We will not activate the publishing job in the github workflow until everything is correct. For some languages the process of publishing a library is quite easy and for some quite annoying (like Dart for example), so I think we need to prepare/investigate that matter as soon as possible.

I applied fixes to kotlin generator ... please check of there are any errors ... as for testing though, i think evchargers, weather and mq are suffice ... no need to test all services

Okay so...

HelloWorld Call and Stream work
the other rather primitive requests should as well
BUT there are more errors regarding some data classes:
The empty ones (for requests and responses). We can not have data classes without content.
Either we leave out empty data classes to solve this problem and to safe storage or we convert them to normal classes without the "data" keyword.

But removing them would also mean that the function with no response don't have a return type.
So you have to somehow be able to remove the response types and request bodies for some of the functions if that's even possible.

you lost me here ... can give an example

Yh of course, the explanation wasn't the best lol :D

https://github.com/GWT-M3O-TEST/m3o-kotlin/blob/30701aa38c1eb42879a203788e3961151d37ab55/src/main/kotlin/com/m3o/m3okotlin/services/analytics/analytics.kt#L43-L54
These are some of the Analytics data classes.
In line 44 there's an empty request class and in line 54 there's an empty response class.
Those kind of empty data classes are the problem.

And let's compare my handcrafted method and the auto-generated one:
suspend fun list(req: AnalyticsListRequest): AnalyticsListResponse {
    return ktorHttpClient.post(getUrl(SERVICE, "List")) {
        body = req
    }
}

You ask for the empty class an an parameter where I drop it as it's obviously not needed because it's empty.
suspend fun list(): AnalyticsListResponse {
    return M3O.ktorHttpClient.post(M3O.getUrl(SERVICE, "List"))
}

It is doable... Will apply a fix soon

Awesome, thanks for your work

Alright, the Analytics endpoints work now.
We can remove the empty request classes now as you correctly removed the request bodies.
Maybe the next step would be removing empty responses.
Example:
suspend fun track(req: AnalyticsTrackRequest): AnalyticsTrackResponse {
    return ktorHttpClient.post(getUrl(SERVICE, "Track")) {
        body = req
    }
}

Here AnalyticsTrackResponse is returned, but the class is empty:
@Serializable
class AnalyticsTrackResponse()

It would then look like this:
suspend fun track(req: AnalyticsTrackRequest) {
    return ktorHttpClient.post(getUrl(SERVICE, "Track")) {
        body = req
    }
}

I just looked through the generator to see how it works and to be able to work on this as well.
Most things I've read makes sense for me, maybe at some point I can work on that on my own.
Then I don't have to let you do the work :)

I just looked through the generator to see how it works and to be able to work on this as well.
Most things I've read makes sense for me, maybe at some point I can work on that on my own.
Then I don't have to let you do the work :)

cool ... actually you should have a full understanding of how the generator works so you can further enhance your m3o-kotlin clients ... maybe m3o-kotlin 2.0 :)
I have applied new fixes ... see if there are any errors

Regarding the request and response classes itself it looks good now.
But it seems we still have the problem that the data types are not correct.
Example: DBMap does not exist
Can you show me again where to fill out the language specific data types?

https://github.com/GWT-M3O-TEST/m3o/blob/ae43f821b2bc209572298a6440ca8bef50702655/cmd/m3o-client-gen/kotlin_generator.go#L65
some how the generator prefix Map with Db
i will check ... you too

I have pushed a fix
see if there are any errors


Fix FF and RW in Automation for Post Mig

Fix Issues for https://smartthings.atlassian.net/browse/CHAD-10342 (POST mig: "Fast forward" and "Rewind" options appear in automation trigger)

Looks good, but as Carter said, wait to merge until after CODA-2736 has been addressed.

Sure, thanks again @greens

LGTM

This looks good.


[IRGen] Change a pass to agree with a merged pair of passes upstream

SanitizerCoveragePass was meerged with ModuleSanitizerCoverage upstream
and thus this usage needs to agree.

Looks fine to me. Do you need someone to merge this?

@vedantk Nope, I have access. Thanks!


Zero assigns

Previously the interpreter had undefined behavior when dealing with assignments with failing guards. Here we explicitly assign those ports 0. Interpreter will also panic if multiple signals drive the same port. However it’s still alright to have multiple assign statements to the same port if each assignment has complementary guards
This is fine:
reg1.in = reg2.write_en ? 1'b1;  reg1.in = !reg2.write_en ? 1'b0
This is not:
reg1.in = reg2.write_en ? 1'b1;   reg1.in = reg1.write_en ? 1'b0;
Because the two guards are not necessarily distinct.
These changes were pretty simple and all contained in interp/src/interpreter/interpret_group.rs , but should be handy!

Forgot to link the initial issue:
#552
Also this probably means we can talk more about:
#542


Add all gallery photo files

I think my gallery photos may have already been add when I re based in the last PR (#5). So I'm not sure if I still need this branch.


feat(foldingrange): add folding range handler

Initial folding range support

VsCode does a thing where is remaps the start and end characters back to the start of the range, but I dont think we need to do that

@neubaner it wont let me request you, but can you take a look pelase


Avoid retrying permanent errors for S3-related services

For example, the S3 client can call the STS service and
receive a forbidden error, which should not be retried.

You could add test to TestPrestoS3FileSystem


Add nix dependencies to stack.yaml?

Consider adding the nix dependencies directly to the stack.yaml file? Nix users can then just use stack --nix build. I noticed pkgconfig is also a dependency, so if you don't want to go this route, perhaps list it as a dependency in the README.
nix:
  packages:
    - pcre
    - pkgconfig

I just ran into this today and was going to submit a PR, thanks for the suggestion!


Enable release build in Windows CI pipelines

Model zoo model tests is only enabled in release build, we should run them.

LGTM Thanks for the fix


Is there a way to make an initial placeholder slide?

Hi. I have the carousel all hooked up and working great.  THANKS a ton for this component!
The last thing I'd like to do is conditionally render a first slide if more than one slide exists.
The line myProps.images[0] === image is an example of what I'm attempting to do.
Is this concept possible?  Thanks!
   <Carousel
    data={myProps.images}
    sliderWidth={Dimensions.get('window').width}
    itemWidth={Dimensions.get('window').width * 0.8 + 8}
    keyExtractor={image => image.id}
    renderItem={({ item: image }) => (
      <View style={myStyles.carouselItemContainer}>
        {myProps.images[0] === image && <Text>THIS IS A SUMMARY SLIDE OF THE SLIDESHOW!! :-)</Text>}
        <ProgressiveImage
          blurredImageUrl={imageUrlThumbnailSized(image.url)}
          finalImageUrl={imageUrlPostSized(image.url)}
          blurRadiusStart={3}
          blurRadiusEnd={1}
          unBlurDuration={1000}
        />
      </View>
    )}
  />

Hi @GollyJer,
Unless I haven't properly understood your need, it should be as simple as the following:
renderItem={({ item: image }) => (
    const child = myProps.images[0] === image ? (
            <Text>THIS IS A SUMMARY SLIDE OF THE SLIDESHOW!! :-)</Text>
    ) : (
        <ProgressiveImage
          blurredImageUrl={imageUrlThumbnailSized(image.url)}
          finalImageUrl={imageUrlPostSized(image.url)}
          blurRadiusStart={3}
          blurRadiusEnd={1}
          unBlurDuration={1000}
        />
    );
    return (
        <View style={myStyles.carouselItemContainer}>
            { child }
        </View>
    )
)}

@bd-arc Thanks for the reply.  I didn't explain myself very clearly... sorry.
As an example... if there are 3 images I want to render 4 slides.
[title slide] [image1] [image2] [image3]
So some way of saying "when there are more than one slide, put an extra title slide at the front".

Since the carousel extends the FlatList component, you can take advantage of the inherited ListHeaderComponent prop. Something along those lines:
const header = data.length >= 3 ? <TitleSlide /> : false;

return (
    <Carousel
      // other props
      ListHeaderComponent={header}
    />
);

@bd-arc thanks so much for the helpful ideas!
It didn't quite work but got a lot closer.
This is a test app with test images...
Here's an example with three slides.

And here is test with a 4th slide "Title" slide added in the ListHeaderComponent prop.
You'll notice the visible depth of the primary slides is lower and I'm unable to scroll to the last slide.

Not sure there's much else to try.
Thanks again for the help you've provided.

@GollyJer Damn, sorry! Of course it can't work since I've never thought about taking the header and footer into account for index handling... The bright side though is that's an update I'm now considering ;-)
For now, do you have any way of altering your dataset if it contains more than two items? For example, you can prepend some specific data for your title item and then render this one conditionally in renderItem().
render () {
    const data = myProps.images;

    if (data.length >= 3) {
        data.unshift({
            type: 'title',
            // other useful data
        });
    }

    return (
        <Carousel
          data={data}
          renderItem={({ item: image }) => (
              const child = item.type && item.type === 'title' ? (
                      <Text>THIS IS A SUMMARY SLIDE OF THE SLIDESHOW!! :-)</Text>
              ) : (
                  <ProgressiveImage
                  blurredImageUrl={imageUrlThumbnailSized(image.url)}
                  finalImageUrl={imageUrlPostSized(image.url)}
                  blurRadiusStart={3}
                  blurRadiusEnd={1}
                  unBlurDuration={1000}
                  />
              );
              return (
                  <View style={myStyles.carouselItemContainer}>
                      { child }
                  </View>
              );
          )}
        />
    );
}

@bd-arc You've given me an idea.  I'll let you know how it goes.
handling the header and footer
This would be a simple way for users to optionally add leading and trailing slides... +1.  😄

Hi @GollyJer,
Any news regarding this issue?

Closing as no further feedback was provided.

@bd-arc Is ListHeaderComponent supported now?

@srameshr Unfortunately no.
I haven't been able to find a way to handle it properly without generating side effects.

I'm trying to understand why this would be problematic.  The main reason I want the header is that the header is not meant to be the same size (using vertical slider, so height) as the other elements.  But I would like the position of each element to end up at the same y-location, which actually it's not doing that either right now, even without a header (there's a slight "drift" in the y-location of each card.

Note:  I was wondering if I should just use a FlatList instead of a RN snap carousel.  I found a blog post that says you can achive a similar result as a carousel with just a few additional properties on the FlatList:
            snapToAlignment={"start"}
            snapToInterval={this.IMAGE_WIDTH + 10}
            decelerationRate={"fast"}
            pagingEnabled

I guess as long as I don't need any fancy effects on the carousel!  So I will have to try this next.

Oh I totally agree: if you don't need any kind of advanced feature or effect, you'll be better off with a regular FlatList, particularly since they made the snapToInterval prop available on Android as well ;-)

Any solution this problem yet ?


Sometimes files downloaded are empty and restart needed

Retrieving files working, but files downloaded are 0-byte size
Expected Behavior
Downloaded files should be correct everytime
Current Behavior
Sometimes I download files but I see they having 0-bytes size. after restarting 0-db and minio things work
Possible Solution
!!
Steps to Reproduce (for bugs)

It's occasional
Context

Just while playing and adding/deleting files for long time
Your Environment

Version used (minio version): Minio with zerostor gateway zerostor branch

Operating System and version (uname -a): Linux hamdy 4.15.0-36-generic #39-Ubuntu SMP Mon Sep 24 16:19:09 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Link to your project: https://github.com/threefoldtech/minio

@Hamdy please try to get a small program that reproduce this everytime

close for incativity


projectiles collision

Is your feature request related to a problem? Please describe.
Projectiles do not collide with walls
Describe the solution you'd like
A simple on/off switch to decide if a projectile collides with walls

yup , we need it.


docs: add example with ws route

Commit Message: A new example using the WebSocket protocol on a specific Route has been added to the documentation and example of working with WebSocket.
Risk Level: Low
Testing: A new example for automated testing has been added to the verify.sh script.
Docs Changes: example.rst has been updated
Release Notes: N/A
Platform Specific Features: N/A

How it looks on site - https://storage.googleapis.com/envoy-pr/6777f12/docs/start/sandboxes/websocket.html
And I don't know why step Envoy/Publish and verify fieled in actions

And I don't know why step Envoy/Publish and verify fieled in actions

i just landed a (hopeful) fix/workaround

/retest

apologies - my fix broke formatting - hopefully should work now


f31 rootless: podman logs: cannot configure rootless cgroup using the cgroupfs manager

Almost certainly related to #4574
$ podman run --name foo alpine date
Mon Feb  3 21:20:10 UTC 2020
$ podman logs foo
cannot configure rootless cgroup using the cgroupfs manager
Mon Feb  3 21:20:10 UTC 2020

It only seems to happen with podman logs, nothing else.
If this requires a new crun, please make sure podman is rebuilt with an updated Requires.
podman-1.8.0-0.1.fc31.x86_64
crun-0.11-1.fc31.x86_64

Issue still present in crun-0.12-1.fc31.x86_64

@giuseppe, can you take a look?

it is just a warning printed by crun, but that doesn't block the execution for the container.  I think it is worth to warn users they cannot use cgroup.

Can we drop this down to Warning, and only display it if the user wants warning messages?

Update: problem does not occur with loginctl enable-linger testuser

Can we drop this down to Warning, and only display it if the user wants warning messages?

the log level is not passed down to the OCI runtime (and there is no way in the OCI specs to do so).
I'll drop the warning from crun.


Running Install-BoxstarterPackage against a remote computer does not allow sourcing a package from a custom NuGet Repo

Migrated from https://boxstarter.codeplex.com/workitem/73
I have my system bootstrap package located in a custom NuGet repo and am running the following command's from my workstation PC against the server I am trying to BootStrap
#################
# Workstation PC
#################

# Set NuGet Repo
Set-BoxStarterConfig -NugetSources "http://www.myget.org/F/MyNugetFeed/api/v2;http://chocolatey.org/api/v2;http://<IP_ADDRESS>:8080/nuget/MyRepo"

# Start Remote Install
Install-BoxstarterPackage -ComputerName <IP_ADDRESS> -Credential $cred -PackageName MyPackageInMyCustomRepo -Verbose

The console then logs the following (I am only showing the important parts)
VERBOSE: [MyServer]Boxstarter: Installing 1 packages from
C:\Users\AdminAccount\AppData\Local\Temp\Boxstarter\BuildPackages;http://chocolatey.org/api/v2;http://www.myget.org/F/boxstarter/api/v2

.
.
.

Chocolatey (v<IP_ADDRESS>) is installing 'MyPackageInMyCustomRepo' and dependencies. By installing you accept the license for
'MyPackageInMyCustomRepo ' and each dependency you are installing.
Unable to find package 'MyPackageInMyCustomRepo'.

Command 'install' failed (sometimes this indicates a partial failure). Additional info/packages: MyPackageInMyCustomRepo
VERBOSE: [MyServer]Boxstarter: Exit Code: 1
Chocolatey reported an unsuccessful exit code of 1
    + CategoryInfo          : NotSpecified: (:) [Write-Error], WriteErrorException
    + FullyQualifiedErrorId : Microsoft.PowerShell.Commands.WriteErrorException,Write-Error
    + PSComputerName        : <IP_ADDRESS>

When BoxStarter is doing this remote installation, it creates the temporary directory C:\Users\MyUserName\AppData\Local\Temp\boxstarter on the destination PC. Inside this folder is boxstarter.config. It is using this file to know where its NuGet soruces are. This is where the problem lies.
I tried putting boxstarter.config on the destination server there first with my custom NuGet repo, but when you run the Install-BoxstarterPackage against the remote server, this file is replaced.
I have also tried pre-installing BoxStarter on the destination server and running Set-BoxStarterConfig but when you run Install-BoxstarterPackage it will only look at the configuration from the C:\Users\MyUserName\AppData\Local\Temp\boxstarter folder.
I am not sure if this is by design or not, but if an additional parameter could be added to the Install-BoxstarterPackage function, for example -UseLocalBoxstarterConfig, this would resolve it. I guess the way to implement is, after sending the remote computer the Boxstarter.zip package and extracting it, also sends the workstations boxstarter.config file to the remote machine, so when the remote machine attempts to search for the package, it uses the same sources as the workstation machine.

I'm still having this issue and it doesn't seem to help to edit the config inside the zip file. Is this being worked on? The "remote installation" part of boxstarter is very flawed when this bug is present.
Keep up the good work!

having the same problem...
I'm on a PS session (on a remote - Azure - computer) by using
$rs = New-PSSession -ComputerName ([Uri]$Service.DNSName).DnsSafeHost -Port $RemotePS.Port -UseSSL:$true -Credential $cr -SessionOption (New-PSSessionOption -SkipCACheck)
and then by invoking script block -
                  Invoke-Command -Session $rs -Scriptblock {
                        choco install boxstarter
                        choco install boxstarter.Azure
                        Set-ExecutionPolicy Unrestricted -Force
                        Import-Module Boxstarter.Chocolatey
                        Install-BoxstarterPackage -Package mssqlserver2012express
                    }

getting this error -
MsSqlServer2012Express v11.00.3000.20140928
ERROR: The system cannot find the file specified.
+ CategoryInfo          : NotSpecified: (ERROR: The syst...file specified.:String) [], RemoteException
+ FullyQualifiedErrorId : NativeCommandError
+ PSComputerName        : eikostest09.cloudapp.net
NotSpecified: (:) [], RemoteException
SqlServer2012Express did not finish successfully. Boo to the chocolatey gods!
[ERROR] Unable to run scheduled task. Message from task was
+ CategoryInfo          : NotSpecified: (:) [Write-Error], WriteErrorException
+ FullyQualifiedErrorId : Microsoft.PowerShell.Commands.WriteErrorException,Write-Error
+ PSComputerName        : xxxxxxxx.cloudapp.net

Package 'MsSqlServer2012Express v11.00.3000.20140928' did not install successfully: Unable to run scheduled task. Message from task was
+ CategoryInfo          : NotSpecified: (:) [Write-Error], WriteErrorException
+ FullyQualifiedErrorId : Microsoft.PowerShell.Commands.WriteErrorException,Write-Error
+ PSComputerName        : xxxxxxx.cloudapp.net
Help!
account is setup to have Admin rights. Does it require to setup some more rights somewhere?
This is definitely related to WinRM, 'cause I don't see such error while doing the same on the machine itself.

Duplicate of #6 and resolved in PR #320


Create and push Microsoft.UI.Xaml vpack as part of Release Build

To have Microsoft.UI.Xaml framework package preinstalled in the OS, we need to create and push a vpack containing the appx so that it can be consumed by the OS.
This was previously done by manually creating and pushing the vpack on an individual's machine, but we need to move this to happen as part of the Pipeline run.
In order for this to happen we need to replace the manual step of Store-signing the framework package with a task in the Pipeline. This PR adds that which simplifies the release process.
We also create and push the Microsoft.UI.Xaml vpack from the Pipeline. This is in addition to the existing CBS vpacks that we also push. The Microsoft.UI.Xaml vpack contains the same framework package as what gets published to the store.

/azp run


Malformed Request with release_unlocked_beta

When I use the "cci flow run release_unlocked_beta --org dev" command, and specify an install key using the install_key yaml option for the create_package_version task, the package version creates, but later, it fails in some automated action that is performed
Error: Malformed request
https://neocol.my.salesforce.com/services/data/v52.0/tooling/query/?q=SELECT+Dependencies+FROM+SubscriberPackageVersion+WHERE+Id%3
D%2704t8W000003JjVhQAK%27. Response content: [{'message': 'Please provide installationkey when querying against
SubscriberPackageVersion that is protected by installationkey', 'errorCode': 'INSTALL_KEY_REQUIRED'}]

The package creation seems to have been successful but there is then some error that follows:
[Success]: Package creation successful
[05/02/23 17:05:50] Exception in task create_package_version

Closing as I opened it in right place: https://github.com/SFDO-Tooling/CumulusCI/issues/3592


dont force windows as a restore state on start

[ ] on windows when the app starts up it should be minimized if auto start but not hidden on an explicit start. maybe need setLogin params and read that out
[x] advanced setting to let you just quit and not go to the tray

Avoiding Investment Scams: A Personal Testimonial
I stumbled upon an enticing investment opportunity on Facebook that promised substantial profits. I started with an investment of $300 and saw a 40% increase in just three days. Encouraged by this success, I invested an additional $5,000.
A week later, I managed to withdraw $2,500, which boosted my confidence. However, as I continued to invest, the company's responsiveness faded. My calls and emails went unanswered, and local police couldn't help as the company was based overseas.
Feeling hopeless, I was contacted by an inspector who recommended Asset Recover Net. I reached out to them and, incredibly, they recovered my entire loss of $290,000 within five days. My portfolio, which had nearly reached a million dollars, was also restored.


ESM1.5 SSP-245-15..-34

resolution_complete type_ScenarioMIP   |    by<EMAIL_ADDRESS>
Description: ssp245 ensemble extension
Suite name(s): SSP-245-15 to SSP-245-34
Output location: /scratch/p66/txz599/archive/
Frequencies: monthly, daily
Sim years: 2015-2100
CMORisation reqd: Yes
(CMOR) CMIP6 exp: ssp245
(CMOR) Parent exp: HI-15..34
(CMOR) Branch years: 2015
(CMOR) ESGF pub: Yes
Other info: N/A

Issue migrated from trac:377 at 2024-01-31 18:35:39 +1100

<EMAIL_ADDRESS>commented

Archival complete: um2netCDF conversion, restarts, archive checker
/g/data/p66/cm2704/archive/SSP-245-{15..34}

<EMAIL_ADDRESS>changed status from new to closed

<EMAIL_ADDRESS>changed milestone from Run complete to Archived

<EMAIL_ADDRESS>set resolution to complete


Better integration guide / example app

Create wallet
Get BTC/ETH/BNB addresses
Sign Tx

It could be a simple C++ cmdline demo app.
We could also make a very minimalistic wallet with Pyton+QT, with wallet creation, show address, make a send.  Minimalistic, but working!

I think people want sample code with description / comment most, no need a full functional example app, and for C++, we already have wallet console to reference

I created a small sample C++ project here:
https://github.com/catenocrypt/sample-wallet-core-cpp
However, I got stuck with signing -- using AnySigner or EthSigner require protobuf input, for which no bindings/headers exist for C++!
The sample (once we solve signing) can be moved to under trustwallet org, or included in wallet-core itself.

no one tries to use wallet core in c++ :)


SparseSet capacity & .NET Standard 2.0 fix

This change removes the initialCapacity increment on construction of SparseSets. This increment caused array out of bounds errors unless you construct it with a capacity 1 lower than your other data-set.
Default assignments are also retyped to be explicit. This makes the SparseSet .NET Standard 2.0 compliant, which I personally needed for some server-side logic.

Makes sense that we move towards making sure everything is .NET Standard 2.0 compliant :) Good catch on the AoB.


Add auto installer for npm and for pm2 for the interactive cli installer (for local installs)

⚠ Please verify that this feature request has NOT been suggested before.

[X] I checked and didn't find similar feature request

🏷 Feature Request Type
Other
🔖 Feature description
While running the auto installer on the Readme, it fails with an error that pm2 couldn't be installed. Doing a bit of digging and following this article listed in the error (https://pm2.keymetrics.io/docs/usage/quick-start/) I found that npm wasn't installed and that the auto installer doesn't attempt to install npm to rectify the issue. Instead the user has to do that installation manually,
I feel it could be beneficial to have it auto install npm and the pm2 package in addition to the other packages it already installs (like node-js). This would ensure that the installer is more hands-off to the user and less user-error prone.
I'd be more than happy to submit a pull request if you'd like that accomplishes the above.
✔ Solution
I envision that you have the installer check for an npm installation before trying to install the pm2 package. If one doesn't exist, go ahead and install npm for the user. Then continue to proceed with the pm2 package installation.
❓ Alternatives
No response
📝 Additional Context
No response

Clarified some language in the feature request.

We have added help texts to the installer in https://github.com/louislam/uptime-kuma/pull/3603
Note that the installer has to run on both linux/mac + windows => installing npm via the guide we now link is eazier and less likely to result in serious/hard to debug defects imo

Do I read you correctly that you have node, but not npm installed?
What was the error message shown?

Maybe get it to run as snaps or similar? UK is one of the least reliable packages on my home debian cluster, not exactly what you'd expect from a monitoring server-package. The dependencies with pm2, npm, node-js, and a reverse proxy are a multitude of points of failure, you'll need an entire system to watchdog and monitor your uptime-kuma.

running the auto installer on the Readme

I think I will remove the installer later, because it is really hard to maintain such things in Linux. Normally npm will be installed along with Node.js, but in some cases, it doesn't.

Maybe get it to run as snaps or similar

May need someone to do that, I am not interested in it. As far as I know, someone made it for Arch Linux (https://aur.archlinux.org/packages/uptime-kuma)

The dependencies with pm2, npm, node-js, and a reverse proxy are a multitude of points of failure, you'll need an entire system to watchdog and monitor your uptime-kuma.

Sorry I don't get what you mean here. Are you expecting a JavaScript application running without Node.js and npm? And are you expecting to "reverse-proxy" without a reverse proxy?
Talk is cheap. If you think you can write this app in a single executable file without dependencies. Just show it to the world.

I've switched to ntopng (the free community edition). May look overwhelming at first, but it's way more sophisticated (first created in 1998), and once you get the hang of adding alerts and pings, it's super logical, and your dashboard is entirely configurable.


Dependancies problem?

I am not sure if this is a bug or not, I am using Linux Mint 18, 64 bit XFCE with Python 2.7.12. You're about box on Debreate says I have Python 2.7.12 and wxPython 3.0.2 on the Sytem information tab.
As a minimal example if I put my dependency to "python=2.7.12" and try a test install it tells me my dependency has not been met, it works on the >= 2.7 though
I cannot get it to recognize any version of wxPython even if I say any wxPython > 2.6 it just says dependency not met. I can send screen shots.
Ken

Could this link help you? https://wiki.wxpython.org/How to install wxPython

The wxPython package is not labelled "wxPython" in Debian/Ubuntu/Mint. It is python-wxgtk2.8, python-wxgtk3.0, or python-wxgtk4.0. Not sure why the python dependency isn't working. Perhaps try "python2.7 = 2.7.12".

I will be closing this soon as I have not received any more responses.

PR #40 indicates more allowed versions for python.


[ios] "Choose My Own Password" not calling onChange

🐛 Bug Report
When you have an input with secureTextEntry the iOS Strong Password dialog pops up and it changes the text to the generated password. This works as expected and onChange is called, but when you press Choose My Own Password (picture below), it doesn't call onChange.
I saw #21261 was closed because it didn't follow the format, and I couldn't find another issue created for this
To Reproduce

Have a <TextInput /> with secureTextEntry on iOS 12+

It is also helpful to have console.log on onChange and onChangeText to see what is called

Tap it, this should bring up the Strong Password popup
Press Choose My Own Password
The text changes visually, but it does not call onChange or onChangeText

Expected Behavior
Pressing Choose My Own Password calls onChange and onChangeText with an empty string. That way we know that the user is no longer using the secure password.
Code Example
https://snack.expo.io/@johnkahn/choose-my-own-password
Cannot be run on the simulator, must be run from an iPhone iOS 12+ with Autofill Passwords turned on in the Settings.

Environment

  React Native Environment Info:
    System:
      OS: macOS 10.14.3
      CPU: (8) x64 Intel(R) Core(TM) i7-7820HQ CPU @ 2.90GHz
      Memory: 41.62 MB / 16.00 GB
      Shell: 3.0.0 - /usr/local/bin/fish
    Binaries:
      Node: 11.10.1 - /usr/local/bin/node
      npm: 6.7.0 - /usr/local/bin/npm
      Watchman: 4.9.0 - /usr/local/bin/watchman
    SDKs:
      iOS SDK:
        Platforms: iOS 12.1, macOS 10.14, tvOS 12.1, watchOS 5.1
    IDEs:
      Xcode: 10.1/10B61 - /usr/bin/xcodebuild
    npmPackages:
      react: 16.5.0 => 16.5.0
      react-native: https://github.com/expo/react-native/archive/sdk-32.0.0.tar.gz => 0.57.1

Thank you for the reproduction. We've confirmed the issue. The issue is marked as mid priority because we want to promote using these security features. A PR to fix this would be much appreciated!

Maybe this can help for making a PR: https://stackoverflow.com/q/53566570/2525941
It seems like that the native delegate
- (BOOL)textField:(__unused UITextField *)textField shouldChangeCharactersInRange:(NSRange)range replacementString:(NSString *)string

is not being called when choosing "Choose My Own Password".

not stale.

Is any of this problem still happening?

yes

Still happens for me. I've tried everything with my forms and still the strong password is a pain to deal with. The only solutions I've found that "work" so far are to add hidden inputs with secureTextEntry so that the app doesn't try and ask for a strong password anymore, but even that is still buggy.
This is also on just basic forms with username and password for textContentType...

I have this problem too

I have this problem too +1

I still have this problem, that none of the standard callbacks are called. Has anybody solved the problem with a work around?

@Luckfried Using react-navigation, I added the following to my Input component:
<NavigationEvents
  onWillBlur={() => setDisableAuthInput(true)}
  onWillFocus={() => setDisableAuthInput(false)}
/>

It will disable any "auth" inputs and make them uneditable when navigating away from the page. I found that when there are several pages in the stack, the strong password will trigger changes on pages in the stack which are not visible.

Have there been any updates on this? This is very problematic!

@johnkahn did you find a workaround?

@LucidNinja I did not, unfortunately. Still an issue for me.

@johnkahn - that's frustrating. I've tried looking into a work around or even writing a patch, but I'm no Objective-C genius!

It looks like this is an issue with iOS's UITextFieldDelegate. Found a similar issue on Stack Overflow for native iOS. It does not fire a change when 'Choose My Own Password' is pressed.

Same problem

You can fix this problem with this solution:
// Add in your TextInput the props "onEndEditing"
<TextInput {...} onEndEditing={event => {
    if(event.nativeEvent.text.length === 0) {
        this.setState({
            password: ""
        });
  }
} />

Now, you can check if the password is empty before submit.

Thanks @JordanLauriol - this workaround seems to have solved the problem!

Awesome! Glad there's a temporary solution. Thanks @donalbertico and @JordanLauriol for the workaround 👍
It's acceptable for my uses until there is an official fix in place

Just wanted to say I feel this issue should be bumped up in priority.
Promoting the use of secure features is awesome but not calling the onChangeText with "" as the value when the user explicitly opts out of using iOS's password generation feature can result is major issues with application internal state.
If my Input field is updating component state when onChangeText is fired, I would expect this event to fire when the user opts out of the iOS secure password feature because the Input field goes from yellow with auto-generated password filled back to it's original styles with no text in the input.
Please reconsider the priority of this and respect Apple's implementation of auto-gen password! I have no way to clear out auto-generated password from component state even after the user explicitly opted out of using an auto-generated password.

Faced with the same issue<EMAIL_ADDRESS>

ой всё

оооо

Not stale, it's still an issue. There is a workaround, but will still cause issues out of the box.

Hey! I am facing the same issue. I want to disable my button if the user still not have enter password. Unfortunately, when user opt for "choose its own password", the Apple strong passwords are stille in the state of my application.

Not stale

We need this! Is that so hard to fix ?

I'm still having this issue with react-native 0.66.4

RN 0.68.1 problem still exists

I am also having this issue RN 0.64

Issue still is present on RN 0.69

Issue still exists! Why this is not treated as higher priority? RN 0.69.5

I am also facing this issue RN 0.70.8

The workaround with onEndEditing kind of works, but if the user manages to press the form submit button before focusing on anything else (i.e. the password input being the last field of the form and the user not pressing the screen to close the on-screen keyboard) the onEndEditing function will not trigger in time, causing the submit function to still have the previous state that includes the password provided by autofill.

I am also having this issue RN 0.68.2

I have the same issue, please fix it :((

I am also having this issue RN 0.72.6

Still happens at 0.73.2.
^This workaround from @aqwxcvb works fine thou.

Not stale, please reopen


Full body talking video?

Thanks for your great work!  May I ask do you have any plans on something  like "full body talking video"  instead of talking head?

i also need this

next work echomimic-v2 will focus on halfbody generation.

next work echomimic-v2 will focus on halfbody generation.

Will this be released this year?


Phase Scalafmt

Description

This is the first attempt to use phase extension. The new phase can format Scala source code via Scalafmt.

It is inserted at the end of customizable phases. The extension adds an additional boolean attribute format to the rule. If format is set to true, users can run
bazel run TARGET_NAME.format

to format the source code, and run
bazel run TARGET_NAME.format-test

to test the formatting (without modifying source code).
Motivation

Formatting is one of the features in higherkindness/rules_scala. I believe it will bring value to this repo.
TODO:

[ ] Write documentation once we agree to move forward.

Thanks for always giving thorough review. I am still not very used to the PR principle here, I will respond to the comments soon.

This PR is >600 LOC

I went through the code again, everything is related and it makes sense to be in the same PR, IMO. But I agree it becomes a big one. I would like to know how to make a PR like this (introducing a new feature), which needs a decent chunk of code to work correctly, small enough for reviewing?
I have thought of separating functions and tests, but does it make sense to merge something without tests? Or merging functions step by step, but does it make sense to merge code which is functionally incomplete?
What do you think? Thanks.

I agree and this is why I wrote that I don't have any concrete feedback here. I wrote it because I wanted to align ourselves about balancing small vs large PRs and also features.
Often we can do small but important features and we should try and prioritize those.
Additionally I'll say that I just opened a PR that doesn't advance us functionally but it's a step in the right direction. The reason I opened it as-is is to help review.
I do agree however it's not exactly the same example and I'm not 100% sure how it translates to this PR (maybe only add format without check, don't know 🤷♀ )


feat: adds support for Xur selling Hawkmoon and Dead Mans Tale

What
feat: adds support for Xur selling Hawkmoon and Dead Mans Tale

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


Update v1.2 help

Modified help such that it redirects users to the User Guide on a browser instead of just copying the URL.

Codecov Report

Merging #40 (78206b9) into master (49fb350) will not change coverage.
The diff coverage is 0.00%.

@@            Coverage Diff            @@
##             master      #40   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

Impacted Files
Coverage Δ

src/main/java/seedu/address/ui/HelpWindow.java
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


Added a section of Adding a page at /docs path

This section answers how one can add a web page at /docs in the app.
Page is added at https://fumadocs.vercel.app/docs/headless/page-conventions which is the same in the ui at https://fumadocs.vercel.app/docs/ui/page-conventions

I prefer moving this to FAQ section under /docs/ui/index.mdx, and please do code formatting beforehand

I prefer moving this to FAQ section under /docs/ui/index.mdx, and please do code formatting beforehand

Wait, I will move this to faq's and open a new pull request after formating


Emergency Response - Hide certain pieces of the Job Posters in the Emergency Response dept

Description
We want to hide certain pieces of the job poster if they're in our new Emergency Response department.
This includes:

Poster close date
job start date
duration
language level
security requirement
classification
Levels of skill for skill requirmenets?
more?

@gobyrne I'll need your help confirming everything we want  to hide.

Looking at the job poster now and it looks like we should hide everything in the basic info section. Missing from your list above:

Location
Remote Work Allowed/Not Allowed
Salary Range (for sure)
Education section
Language Requirements Section
Maybe the whole Team Culture section... will depend on what we here about the "needs" of departments though.

Link to prototype:
https://xd.adobe.com/view/3f21a885-7176-4329-7b17-8142b1939365-4910/
Its still very rough, will refine over the next day

https://xd.adobe.com/view/e966ee5a-d99c-44ef-6df5-51548178b473-9f57/screen/dafc479a-33bc-487c-aeb3-418012c7bd83/Talent-Cloud-Job-Poster-1?fullscreen


configuration: document use of PAT and no longer log the username

A PAT is simply set as a username. In order not to leak this in the
Ansible logs, no_log must be set for all tasks that use the username.
Closes #308
Signed-off-by: Christian Berendt<EMAIL_ADDRESS>

Fast Forward. Need this in preparation for the dataport testbed.


[3.8] Fix typo on the HTTP Async Transport page

Same as https://github.com/quarkiverse/quarkus-cxf/pull/1372#pullrequestreview-2049634292

Same as https://github.com/quarkiverse/quarkus-cxf/pull/1372#issuecomment-2171429858


Minimize to tray support

Should the tray icon go away when the window is re-opened?

Also, this change is conflicting with a change made in another PR: https://github.com/Storj/driveshare-gui/pull/51
Please make sure to coordinate with other devs on who's supposed to do what on Jixee to avoid duplication of efforts. Talk to Shawn if there is a workflow issue that needs to be addressed.

@gordonwritescode This pull should be on the develop branch.


Cleanup sometimes run in the wrong directory

runner@gh-actions-runner:~/_work$ ls -alh
total 32K
drwxr-xr-x  8 runner runner 4.0K Apr 29 22:43 .
drwxr-xr-x 10 runner runner 4.0K Apr 29 21:57 ..
drwxr-xr-x  5 runner runner 4.0K Apr 29 22:41 _actions
drwxr-xr-x  3 runner runner 4.0K Apr 29 22:41 bambi-service-stonksexchange
drwxr-xr-x  3 runner runner 4.0K Apr 29 22:41 _PipelineMapping
drwxr-xr-x  3 runner runner 4.0K Apr 29 22:43 .pytest_cache
drwxr-xr-x  2 runner runner 4.0K Apr 29 22:43 _temp
drwxr-xr-x  3 runner runner 4.0K Apr 29 22:41 _tool
runner@gh-actions-runner:~/_work$ ls -alh bambi-service-stonksexchange/bambi-service-stonksexchange/
total 8.0K
drwxr-xr-x 2 runner runner 4.0K Apr 29 22:43 .
drwxr-xr-x 3 runner runner 4.0K Apr 29 22:41 ..

it appears that sometimes the cleanup step is being run while in the wrong working directory
https://github.com/enowars/bambi-service-stonksexchange/runs/2470470218?check_suite_focus=true

~/_work/bambi-service-stonksexchange/bambi-service-stonksexchange/ is the one that should be cleaned 🤔
~/_work should stay untouched, it is also used for caching actions and tools.

Ah, ok. I just assumed that those directories should be cleared as well, since pip was using cached packages instead of fetching the newest versions. But after giving this some more thought, I think pip does actually search for the current version first and only uses the cached one if there is no newer version available, so this should not cause issues


Bump cargo_metadata version

At work we're unable to update cargo-metadata to the version released recently due to our clap-cargo dependency. So I tried updating it.
Tests pass, but I haven't looked in-depth to see if there's any reason not to just do this (well, it would need a 0.11 release of course).

Hmm, RenovateBot should have made this PR.  I've poked them about this: https://github.com/renovatebot/renovate/discussions/23642

New release out

Thanks!


ERROR: error loading package 'bazel-compilation-database/tests': Extension file not found. Unable to load package for '@compdb//:aspects.bzl': The repository could not be resolved

➜  hello-server git:(master) ✗ git clone git@github.com:grailbio/bazel-compilation-database.git
Cloning into 'bazel-compilation-database'...
remote: Enumerating objects: 89, done.
remote: Total 89 (delta 0), reused 0 (delta 0), pack-reused 89
Receiving objects: 100% (89/89), 24.38 KiB | 4.88 MiB/s, done.
Resolving deltas: 100% (47/47), done.
➜  hello-server git:(master) ✗ ls
bazel-bin  bazel-compilation-database  bazel-genfiles  bazel-hello-server  bazel-out  bazel-testlogs  BUILD  CPPLINT.cfg  src  tags  tools  WORKSPACE
➜  hello-server git:(master) ✗ ./bazel-compilation-database/generate.sh
ERROR: error loading package 'bazel-compilation-database/tests': Extension file not found. Unable to load package for '@compdb//:aspects.bzl': The repository could not be resolved
INFO: Analysed 0 targets (0 packages loaded).
INFO: Found 0 targets...
INFO: Elapsed time: 0.719s, Critical Path: 0.00s
INFO: 0 processes.
➜  hello-server git:(master) ✗

git clone will not work because it will also copy the tests directory which will not work.
The instructions given in the README will work for you. Which suggests to directly download a release archive. These archives exclude the tests directory through the .gitattributes file.
Please close the issue if it works for you.

thanks, I download a release archive, then appear another problem
ERROR: Invalid aspect './aspects.bzl%compilation_database_aspect': Invalid label: invalid package name '.': package name component contains only '.' characters: Invalid label: invalid package name '.': package name component contains only '.' characters
INFO: Elapsed time: 0.233s
INFO: 0 processes.

git clone will not work because it will also copy the tests directory which will not work.
The instructions given in the README will work for you. Which suggests to directly download a release archive. These archives exclude the tests directory through the .gitattributes file.
Please close the issue if it works for you.

I have a test demo on github ,do you need it ?

fix it by add a build rule to BUILD

fix it by add a build rule to BUILD
Sorry to bother you . I have got the same problem , could you describe  the solve method in more detail please？ Thanks a lot.

fix it by add a build rule to BUILD

Sorry to bother you . I have got the same problem , could you describe the solve method in more detail please？ Thanks a lot.

Hello @Abbyyan, can you tell me how are you using the project? Are you using the generate.sh script, the YouCompleteMe python file, or the bazel build rule?


Error logging method does not logs Error instance correctly

Please tell us about your environment:

winston version? 3.1.0

[ ] winston@2
[x] winston@3

node -v outputs: v8.11.3

Operating System? Linux

Language? all

What is the problem?
logger.error called with Error instance doesn't use message and stack fields, instead logs test error.
Code example:
const winston = require('winston');

const alignedWithColorsAndTime = winston.format.combine(
  // winston.format.colorize({ all: true }),
  // winston.format.timestamp(),
  // winston.format.align(),
  winston.format.printf((info) => {
    const { level, message, ...args } = info;

    if (level.indexOf('error') !== -1) {
      console.log(typeof message, JSON.stringify(message));
    }

    return `[${level}]: ${message} ${Object.keys(args).length ? JSON.stringify(args, null, 2) : ''}`;
  })
);

const transports = [new winston.transports.Console({
  level: 'debug',
  format: alignedWithColorsAndTime
})];

const logger = winston.createLogger({
  level: 'debug',
  transports
});

try {
  logger.info('aaaaa');
  logger.debug('bbbb');
  logger.error('eeee');

  throw new Error('Scary error');
} catch (error) {
  logger.error(error);
}

What do you expect to happen instead?
At least actual error message logged. Ideally - with stack.

It doesn't depends on printf formatter, simple one gives the same result. Event without formatter it returns empty message.

Probably related issue #1422

Had same issue, I had to implement logger.error myself:
  logger.error = err => {
      if (err instanceof Error) {
        logger.log({ level: 'error', message: `${err.stack || err}` });
      } else {
        logger.log({ level: 'error', message: err });
      }
    };

@gtsec actually I did almost the same just used new method name .exception.

This might be caused by wintson-transport, see https://github.com/winstonjs/winston-transport/issues/31 for the issue and https://github.com/winstonjs/winston-transport/pull/34 for the PR.

@DABH why did you closed this issue? Using of formatter suggested by @jeremy-j-ackso does not works without format.json() formatter. It is not even workaround.
const winston = require('winston');
const { format } = winston;

const logger = winston.createLogger({
  format: format.combine(
    format.errors({ stack: true }),
    format.metadata()
  ),
  transports: [ new winston.transports.Console() ],
});

function a() {
  throw new Error('Error from "a"');
  console.log('a()');
}

function b() {
  a();

  console.log('a() called from b()');
}

function c() {
  b();

  console.log('b() called from c()');
}

function f() {
  try {
    c();

    console.log('c() called from d()');
  } catch(error) {
    logger.error(error);
  }
}

f();

$ node 1.js
Error from "a"

Tested on<EMAIL_ADDRESS>

Yeah, I agree with @nosuchip - there doesn't seem to be a good solution if you want to use the simple() format. We actually need both to work in a project I have. I ended up creating a custom formatter which works but is a bit of a hack. This code gives me a stack in json when using that format and a stack in raw text when using simple format:
const displayStack = winston.format((info) => {
  info.message = info.stack ? info.stack : info.message
  return info
})

const consoleDebugFormat = winston.format.combine(
  displayStack(),
  winston.format.simple()
)

const jsonFormat = winston.format.combine(
  winston.format.errors({ stack: true }),
  winston.format.json()
)

const activeFormat = process.env.LOG_FORMAT === 'simple' ? consoleDebugFormat : jsonFormat

const loggerConsole = new winston.transports.Console()
const logger = winston.createLogger({
  transports: [loggerConsole],
  format: activeFormat,
  exitOnError: false
})

So there is still not a solution to this issue?

@cogscape unfortunately no. Just tested latest winston and no one solution found over internet gave in output error message and stack. Except custom method, of course.

For TypeScript I am using following snippet to avoid typing warnings:
...
// standard transports and format initialization

export interface BetterLogger extends winston.Logger {
    exception: (error: Error, prefix?: string) => BetterLogger;
}

const logger: BetterLogger = winston.createLogger({
    level: config.logLevel,
    transports,
}) as BetterLogger;

// Monkey patching Winston because it incorrectly logs `Error` instances even in 2020
// Related issue: https://github.com/winstonjs/winston/issues/1498
logger.exception = function (error, prefix?) {
    const message = error.message || error.toString();
    const stack = error.stack;
    prefix = prefix ? `${prefix} ` : '';

    return this.error(`${prefix}${message}, stack ${stack}`) as BetterLogger;
};

export default logger;

It works with any other formatters.
Reason: attempt to implement current formatter always ends with fail as info passed to formatter never instanceof Error.

I can't log any meaningful error:
import winston from 'winston';
const consoleTransport = new winston.transports.Console({
    format: winston.format.combine(
        winston.format.errors({ stack: true }),
        winston.format.metadata(),
        winston.format.json(),
    )
});
export const logger = winston.createLogger({
    transports: [
        consoleTransport,
    ]
});

It only logs this sad bit:
{"level":"error","metadata":{}}
just using simple is almost better as it produces double the information:
error: undefined
What I need:
    at Object.updateSession (U:\Users\muj\endpoint\src\services\cookieService.ts:88:23)```

is there a way to get via winston the information that node print automatically when it catches unhandled error?

Any updates?

so basically, if I want the stack trace I have to either

use errors formatter with json and meta formatters and get back a json formatter log
or implement my own custom formatter to give me simple text log output
I am correct or am I missing anything ??

It's 2021 and still didn't work for Error instances. I'm stopped using this.

This is an up, as I think this should be taking account.
I upgraded winston from 2 to 3, but I am now downgrading to 2 because of this.

This is a very critical issue for our app, we use winston and when errors occur there is no way to see the stack

This is a very critical issue for our app, we use winston and when errors occur there is no way to see the stack without tinkering the .error

I have the same problem

I expect this to work out of the box without a formatter.
Can we get someone to merge @Ayzrian's fix please?

+1,, would really like to see a fix merged for this issue. Error handling with stack trace option seems relatively worthless if you can't create a log that contains an error message and stack trace.

My fix for now, pretty ridiculous:
const logger = isProdEnv ? winstonLogger : console;

logger.error("Hi")
logger.error("Hello")
This is printing only first line in error file by using winston. Why?
Whereas logger.info prints all logs.

See my comment here : https://github.com/winstonjs/logform/issues/100
This is an internal bug inside base Transport class, that needs to be adressed.

Just in case I'm using metadata to log errors, not ideal but it works.

in 2023 the most popular node.js logging library with 10 million weekly downloads can't log Error instances by default? I'm not even getting a message, just "undefined", nice

in 2023 the most popular node.js logging library with 10 million weekly downloads can't log Error instances by default? I'm not even getting a message, just "undefined", nice

It is open source... You could contribute!

Any update on this?


Enable support for Node & Yarn installation

Summary
This PR adds support for detection of a package.json or yarn.lock, which will cause the following buildpacks/dependencies to be required as part of the build:

yarn.lock - requires install of yarn and node dependencies (can be provided by the yarn & node-engine buildpacks)
package.json - requires install of node dependency (can be provided by the node-engine buildpack)

The Javascript package manager file should be specified with the environment variable BP_JS_PACKAGE_MANAGER_FILE, which expects a path relative to your application root and points to a package.json or yarn.lock file. This variable defaults to "", therefore requiring explicit opt-in to this feature.
This will require both the Yarn and NodeEngine buildpacks to be added to the Java composite order group as optional.
Use Cases
Enables projects which build frontend/javascript code to run commands such as 'yarn' to build alongside Java code. See paketo-buildpacks/java#651.
Checklist

[ ] I have viewed, signed, and submitted the Contributor License Agreement.
[ ] I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
[ ] I have added an integration test, if necessary.
[ ] I have reviewed the styleguide for guidance on my code quality.
[ ] I'm happy with the commit history on this PR (I have rebased/squashed as needed).

Going with #216


Support Redis Sentinel

You should avoid setting your own connection properties to redis and simply use the redis plugin to manage your connection.  This way getting support for sentinel and property file support for sentinel will be inherited.

@ctoestreich spring-session uses spring-data-redis project to connect with Redis instance. Adding redis plugin to manage redis connection won't help that much. Spring-data-redis provides sentinel support itself.


docker 0.29.0-pytorch-inf2 with meta-llama/Meta-Llama-3.1-8B-Instructn failes

Description
Unable to use open-ai endpoint, getting the error below.
Error Message
PyProcess W-100-model-stdout: The following parameters are not supported by neuron with rolling batch: {'frequency_penalty'}.
How to Reproduce?
Using Docker.
"image": "deepjavalibrary/djl-serving:0.29.0-pytorch-inf2"
"envVars": "AWS_NEURON_VISIBLE_DEVICES=ALL
OPTION_TENSOR_PARALLEL_DEGREE=max
HF_HOME=/tmp/.cache/huggingface
OPTION_MODEL_ID=meta-llama/Meta-Llama-3.1-8B-Instruct
OPTION_ENTRYPOINT=djl_python.transformers_neuronx
OPTION_TRUST_REMOTE_CODE=true
SERVING_LOAD_MODELS=test::Python=/opt/ml/model
OPTION_ROLLING_BATCH=auto
OPTION_ENABLE_CHUNKED_PREFILL=true
OPTION_MAX_ROLLING_BATCH_SIZE=32
OPTION_N_POSITIONS=8192
OPTION_MAX_BATCH_DELAY=500
DJL_CACHE_DIR=/tmp/.cache/ ",

PyProcess W-100-model-stdout: The following parameters are not supported by neuron with rolling batch: {'frequency_penalty'}. This is just a warning. We would not fail because of this.
Do you have any other error messages in the log?


Add iputils to all images to allow the use of ping

At present /bin/ping is a symlink to busybox
lrwxrwxrwx    1 root     root            12 Feb 10 16:45 ping -> /bin/busybox
lrwxrwxrwx    1 root     root            12 Feb 10 16:45 ping6 -> /bin/busybox

You need to be root in order to execute tools like ping.
According to https://serverfault.com/a/1001312/54411, we can add the iputils package, and then prefer that for the ping command. Then non-root can execute ping.
Ping is useful for many things, from measuring latency to testing connectivity.

SUID binaries will not work with the restricted pod security standard policy (see the section on privilege escalation).
Instead an admin can just run a pod on the same node manually for checking latency between the node and other services.
If you want to add the ability to ping from all Lagoon workload pods then since ping_group_range is in the safe set of sysctls you could add it to the standard template for Lagoon pods.


Participant boilerplate comparison

As per https://github.com/metanorma/metanorma-ieee/issues/101#issuecomment-1185625112
@ronaldtse I see four documents on mn-samples-ieee-private:
For master-1-standard:

For master-2-standard:

For P2830_d1:

For 2830-2021:

The first one is a guide, and the other ones are standards.
Maybe the difference resides on the stage of the document?

We are allowing all the text to be user-supplied, other than the initial sentence of each participants subsection. @manuelfuenmayor @ronaldtse If we must make that be user-supplied as well, we will, but I'd really rather avoid that. Currently we are using the wording of master-1-standard

If we must make that be user-supplied as well, we will, but I'd really rather avoid that.

I don't think we need to make it user-supplied. I will seek IEEE clarification.

The sentence in master-1-standard uses an identical pattern as 2830-2021, differing at the document type.
P2830_d1 and 2830-2021 sentences differ where the word "IEEE" is substituted as "draft" for the draft.

I think these are the only two things we need to handle.


Remove BatchedNbtStorage

Literally no one uses this.

For context, BatchedNbtStorage was added in an attempt to somewhat replicate how Minecraft stores information, particularly player and world information, but also how player settings stores its data, though that serialises to YAML. The problem with this is, again, the assumptions made with the abstraction layer: it assumes that everything ought to be loaded as once. And depending on what's being stored and how much there is, this could be like force-loading each and every player to be loaded from storage, blocking..ly, or even every chunk in a world. And while this class could be easily modified to be less awful, particularly since it's not used, so there's no risk of breaking changes, it's probably a good thing to yeet this regardless, particularly since https://github.com/CivMC/CivDocker/issues/190 and the virtual threads that would come with it, would largely make this abstraction useless anyway.


Fixed link compatibility with the specification

Links that are simply strings were causing parsing errors in the
JSON deserialization. This is allowed in JSON:API v1.
The top-level links object in Document was of the wrong type. It
should be *Links not *Link.

Can you please add test cases to ensure this is working as expected for the case(s) that made this a requirement? Thanks for your contribution.

Corresponding documentation for reference: http://jsonapi.org/format/#document-links. I guess I'll dig into this and add some tests. Not comfortable merging until then.

I have been meaning to get back to this, but I haven't had time yet. I will get it on my radar again and I hope to contribute the tests soon.

Closing in favor of #31.


19.0.0 tasks

"NIFM" works as is.
"ES" works as is.
"FS" regex offset 2 needs to be investigated.
"NIM" regex needs to be investigated.

preliminary findings for FS offset 2:
function has 3 notable changes.
expecation of zero before existing check.
expecation of zero a new second location within same array
expecation of zero as before within same array (old behavior still present)


Don't fetch maven task from the generic pr example

And reword/reformat other templates
Signed-off-by: Chmouel Boudjnah<EMAIL_ADDRESS>
Changes

Submitter Checklist

[ ] ♽  Run make test lint before submitting a PR (ie: with pre-commit, no need to waste CPU cycle on CI
[ ] 📖 If you are adding a user facing feature or make a change of the behavior, please verify that you have documented it
[ ] 🧪 100% coverage is not a target but most of the time we would rather have a unit test if you make a code change.
[ ] 🎁 If that's something that is possible to do please ensure to check if we can add a e2e test.
[ ] 🔎 If there is a flakiness in the CI tests then don't necessary ignore it, better get the flakyness fixed before merging or if that's not possible there is a good reason to bypass it. (token rate limitation may be a good reason to skip).

/retest

yep sometime locally it doesn't fail but fails in CI 🤔

works now :-D
i had to fix the unittests,
pre-commit didn't work here, because it detect change on .go files but we had changed in yaml files here


[CWS][SEC-12638] Add FIM support on windows

What does this PR do?
This PR aims to add FIM support on windows. The current supported events are file creation, registry creation, open, deletion and setting value of a registry key

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes
In a windows environment, enable FIM:
In system-probe.yaml
runtime_security_config:     fim_enabled: true
Create a file or a registry and check event in the app

Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided. Except if the qa/skip-qa label, with required either qa/done or qa/no-code-change labels, are applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

/merge


Bulk Email using Python

Describe the project you want to add with tech stack
Bulk Email using Python

plz assign me this task under the hacktoberfest label. @pranjay-poddar


Add present() method to filter and extract Optional values

I find myself doing this a lot when working with streams of optionals:
someStreamOfOptionals
    .filter(Optional::isPresent)
    .map(Optional::get)
    ...

This boilerplate seems common enough that it would be convenient to have a present() method:
someStreamOfOptionals
    .present()
    ...

Thoughts?

To give a real-world example of some code I'm trying to improve:
		return StreamEx.of(shop.getLanguages())
				.map(Language::valueOfOpt)
				.filter(Optional::isPresent)
				.map(Optional::get)
				.map(lang -> getListingTranslation(shopId, listingId, lang))
				.filter(Optional::isPresent)
				.map(Optional::get)
				.toImmutableList();

can you use java 9?

Not yet (for a variety of Google App Engine related reasons). But in Java 9, what's the idiom that cleans this up?

in j9 there is Optional.stream so you can do
someStreamOfOptionals.flatmap(optional::stream)

Adding .present() is not very good idea, as this method will be available for any stream, not just stream of optional, and its meaning will be confusing. People may accidentally call it expecting some different behavior. The same behavior is already provided with .mapPartial(fn) method, so you can use
streamOfOptionals.mapPartial(x -> x) // or use Function.identity()

Another (though possibly less performant) way to achieve this is to use StreamEx.of(Optional):
streamOfOptionals.flatMap(StreamEx::of)

This solution could be useful if the streamOfOptionals is not StreamEx object but a classic JDK stream.

Your bigger sample could be reduced to:
    return StreamEx.of(shop.getLanguages())
        .mapPartial(Language::valueOfOpt)
        .mapPartial(lang -> getListingTranslation(shopId, listingId, lang))
        .toImmutableList();

Looks like mapPartial is exactly what I am looking for, thanks!


Make internal snapshots generation based on experimental snapshot property

Current dependencies on/for this PR:

main

PR #92   👈

This comment was auto-generated by Graphite.


Context-menu applies same color to all objects (npcs, items, etc.)

Do you want to request a feature or report a bug?
bug
What is the current behavior?
The context-menu color applies same color to the same item on the menu (orange).

If the current behavior is a bug, please provide the exact steps to reproduce.
Find where an item and an NPC are in the same square. Right-click to open context-menu.
What is the expected behavior?
The NPC (Baynard) should be in yellow text instead of orange.


fix ceph playbook task order: packages need to be installed first

No, they don't, all those actions are run locally on the machine running Ansible.
On Debian, that's the ceph-base and ceph-common packages which would need to be installed on the machine running Ansible.
Definitely something that should be mentioned in the README :)
We could have Ansible try to perform the package installation itself but I don't really like that as I don't want Ansible to start modifying the system it's running on.

Thank you for the hint. I will try it out and add it to the README. Closing this as it is based on a false assumption.


Still alive?

Is js.ipfs.io  still alive and being worked on?

Same question.

Very much so - is there something specific you noticed or had an idea to improve?


change mranderson048 v2 to v6

As discussed here https://github.com/jacekschae/shadow-re-frame/issues/1
when building a project
[:app] Build failure:
The required namespace "mranderson048.re-frame.v0v10v2.re-frame.core" is not available, it was required by "day8/re_frame_10x/view/components.cljs".

I swapped the name of the ns from v2 to v6
Hope this fixes the issue, not sure how to run this locally.

Thanks, I've deployed 0.3.5 and 0.3.5-react16. Sorry about this.

No problem at all. I am glad I was able to help.


Outdated information?

I cannot find this menu on my OneDrive account page. The provided link above https://admin.onedrive.com/?v=SyncSettings goes to https://onedrive.live.com and I cannot find the relevant link to access sync settings (e.g.  (https://onedrive.live.com/?v=managestorage). Please update this information.

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

ID: a3a0cebd-7e82-6631-bdee-031b3b84f74d
Version Independent ID: ee04db29-a6db-2874-2573-d80f0b415308
Content: Block syncing of specific file types
Content Source: OneDrive/block-file-types.md
Service: one-drive
GitHub Login: @kaarins
Microsoft Alias: kaarins

@poweroftao  Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

@officedocsbot assign @scanum

The link takes you to the right location in the OneDrive for Business admin center. Perhaps you are looking at OneDrive personal account?

@poweroftao  We appreciate any feedback that improves the content in the Microsoft docs but we haven't had any response from you so we are going to close this issue.

Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.

@officedocsbot close

Hello, I also met the same problem.
When I open https://admin.onedrive.com/?v=SyncSettings, I actually entered https://onedrive.live.com. I didn't find a similar option in the settings. I hope to ignore one. File or file type to ensure that onedrive has no exception prompt.
I am an office 365 Home Edition user.

@mikewaretena Could you please provide more information about personal onedrive?


Updating/work-with this template for GHA and Azd Devcontainer feature.

We are updating this template to support new features in Github Action and Azd Devcontainer.

For the Github Action feature: We will remove two lines of code on container image and add GHA code after checkout step in the .github/workflow/azure-dev.yml file. (Shown as below)

For the Azd Devcontainer feature: We will remove dockerfile in .devcontainer folder and add  the following code in devcontainer.json file,

"ghcr.io/azure/azure-dev/azd:latest": {}

and image also need to be add instead of build. Such as:

@rajeshkamal5050 , @crazy4pi314 for notification.

I have made these changes and will push them shortly, thanks for the heads up!


Move metrics and health checks to separate port

This PR moves metrics and health checks to a separate port.
Fixes #297

@bittrance @phillebaba @NissesSenap no review required, just an fyi.

Coverage increased (+0.04%) to 90.699% when pulling 681cd3a6dbc5ef2d9565c2d1d9e9f43406b64262 on metrics-port into a9d64806f8e4bfe7d925f1b32bb4a159b3cdd383 on main.

Don't forget to update Terrform to match the new metric port, it would be needed for the servicemonitor

Don't forget to update Terrform to match the new metric port, it would be needed for the servicemonitor

I wouldn't dream of it!


[Stable/Kong] Migration on update job always failing when installing with ingress controller

Is this a request for help?: Yes

Is this a BUG REPORT or FEATURE REQUEST? (choose one): BUG REPORT

Version of Helm and Kubernetes:
Client: &version.Version{SemVer:"v2.10.0", GitCommit:"9ad53aac42165a5fadc6c87be0dea6b115f93090", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.10.0", GitCommit:"9ad53aac42165a5fadc6c87be0dea6b115f93090", GitTreeState:"clean"}
Client Version: version.Info{Major:"1", Minor:"12", GitVersion:"v1.12.2", GitCommit:"17c77c7898218073f14c8d573582e8d2313dc740", GitTreeState:"clean", BuildDate:"2018-10-30T21:39:16Z", GoVersion:"go1.11.1", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"12", GitVersion:"v1.12.1", GitCommit:"4ed3216f3ec431b140b1d899130a69fc671678f4", GitTreeState:"clean", BuildDate:"2018-10-05T16:36:14Z", GoVersion:"go1.10.4", Compiler:"gc", Platform:"linux/amd64"}
Which chart:
stable/kong
What happened:
when installing kong with ingress controller, there are 2 migration jobs, the one names migration-on-upgrade always fails and thus helm install fails. The migration job's pod result in this error
Error: [postgres error] could not retrieve current migrations: [postgres error] FATAL: password authentication failed for user "kong"

I checked the secret too it contains the password. But when I try to do psql from a pod with the password in secret it fails.
What you expected to happen:
This job should not fail and the installation should work fine. The password in secret should be correct.
How to reproduce it (as minimally and precisely as possible):
Install helm with ingress controller, first time it works fine and later on when next deployment is done it start failing. After deleting the helm release and deploying helm works fine.
Anything else we need to know:

I think both the migration scripts are same so another question is do we need both the migration scripts?

@prakharjoshi One migration Job is used at for the migrations at the start to bootstrap and run the initial set of migration.
The other one is used with a upgrade hook, meaning that the migration runs every-time you upgrade Kong or the helm chart.
Regarding,

Install helm with ingress controller, first time it works fine and later on when next deployment is done it start failing. After deleting the helm release and deploying helm works fine.

Could you elaborate on what you mean by "when next deployment is done it start failing"?
Are you trying to deploy another Kong app in the same cluster or something else?

@hbagdi It was the issue with miss configuration for Postgres chart, I was not defining the password for Postgres and every deployment the secret created in the chart was creating a new password and hence was causing failure for migration jobs as the jobs are picking password from secret.
I guess an update in doc will be handy here to avoid such situations :)

PR welcome!
On Wed, Dec 5, 2018, 4:19 AM prakhar<EMAIL_ADDRESS>wrote:

@hbagdi https://github.com/hbagdi It was the issue with miss
configuration for Postgres chart, I was not defining the password for
Postgres and every deployment the secret created in the chart was creating
a new password and hence was causing failure for migration jobs as the jobs
are picking password from secret.
I guess an update in doc will be handy here to avoid such situations :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/helm/charts/issues/9543#issuecomment-444466271, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AEeiOTfEwHg8pTNrksYBwnL7jQB_r2YWks5u17nOgaJpZM4Yy_Bf
.

It's still an issue, just running:
helm upgrade kong stable/kong \
                --install \
                --set ingressController.enabled=true

Twice will fail.

@rafalwrzeszcz
I had the same issue, it comes with the fact that doing helm upgrade kong, it will also do an upgrade of postgresql.
Best way to debug is to compare outputs of:
helm upgrade -f values.yaml kong stable/kong --dry-run
helm get kong
You will see that the section with kind: Secret is changed.
Quick fix is to make sure that Helm upgrade is producing the same "Secret" as the previous release. One way to do it is:
helm upgrade -f values.yaml --set postgresql.postgresqlPassword=XXXX kong stable/kong
https://github.com/helm/charts/issues/6859
https://github.com/helm/charts/issues/5167
PS: Using postgresql.existingSecret does not work, since it won't produce a resource of kind: Secret, which will result in deleting the secret on K8

Thank @SeriousJul

Quick fix is to make sure that Helm upgrade is producing the same "Secret" as the previous release. One way to do it is:
helm upgrade -f values.yaml --set postgresql.postgresqlPassword=XXXX kong stable/kong

PR welcome to document this in the chart itself.


⚠ T:Riza Docs has degraded performance

In aa497fd, T:Riza Docs (https://docs.triza.dev) experienced degraded performance:

HTTP code: 200
Response time: 3413 ms

Resolved: T:Riza Docs performance has improved in c904db6.


10173 Misc date cleanup

Issue #: /bcgov/entity#10173
Description of changes:

renamed createUtcDateFromStr -> yyyyMmDdToDate
renamed dateStrToPacificDate -> yyyyMmDdToPacificDate
deleted unused date mixin variable and method
added date-mixin test file
added legal-api-mixin test file
added mixin-tester
misc cleanup
app version: 3.0.30

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the bcrs-entities-create-ui license (Apache 2.0).

Ready for review. This is mostly clean / sync methods with other UIs. Should be safe to merge/deploy.


Gcash

Thanks for looking to open an issue for this project.
If you are opening an issue to suggest adding a new entry, please consider opening a pull request instead!

Bzzz

What

My account xan187 is compromised over the past month everything has not been me. Ppl/hackers have email Apple ID and iTunes. I'm turn to stop it but it could be internal. With info like ssn. Please don't help on anything posted with this account unless directed towards information retrieval! I will continue to look at this and email for info if anyone can help

My account xan187 is compromised over the past month everything has not been me. Ppl/hackers have email Apple ID and iTunes. I'm turn to stop it but it could be internal. With info like ssn. Please don't help on anything posted with this account unless directed towards information retrieval! I will continue to look at this and email for info if anyone can help

Thanks


[FEATURE] Maximum size for images or image processing option

Is your feature request related to a problem? Please describe.
Define a maximum size for images, to avoid having really big images due to large upload times.
Describe the solution you'd like
I would like to either have a direct option to set a maximum image size (If an image with a bigger size it uploaded it should be compressed to the specified maximum size) or handle image insertions manually. With the mediaUploadInterceptor  I can accept or decline images, but I can not process them and compress them manually.

You can create a custom image upload button to control the entire uploading process, this should allow to do what you want.


MINOR: Upgrade rocksdb to 5.14.2

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@mjsax @bbejeck @vvcephei

Can we run system tests?

System test passed: https://jenkins.confluent.io/job/system-test-kafka-branch-builder/1841/


Break up state.rs

Stuck on a plane with no in-flight entertainment, so decided to make some quality-of-life improvements.

Split out parts of state.rs into a new repo.rs
Renaming and comments to improve readability
Moved natural language query parsing into the semantic search pipeline

the rest of the refactor looks good! i do like the new nomenclature, and repo::RepoMetadata feels more natural than state::RepoHeadInfo.


Flow Builder triggers completely gone and breaking all flows (community edition)

PHP Version
8.2.8
Shopware Version
6.5.4
Expected behaviour
When I visit the Flow Builder in SW-Admin, I usually always had the option to add new flows with triggers and edit existing flows.
Actual behaviour
When visiting the Flow Builder page in SW-Admin, and creating a new flow or editing an old flow, there are no more triggers available at all. There is an error message "Elements can only be created in the system default language..." which you can only read if you look into the source code, because it's truncated in admin.
I am already using default language AND I have all admin rights.
Error in console is An error was captured in current module: TypeError: Cannot read properties of undefined (reading 'treeItems')

Previously it was always possible to select the respective trigger, but now the triggers are empty. Changing existing flows is also no longer possible. All previously existing triggers are no longer found.

For more information, see my forum entry.
How to reproduce
I am not quite sure, but this problem has been occurring in our system since the update to SW <IP_ADDRESS>.

Update: The bug is caused by the B2B Workflow plugin from Viosys. I am in contact with the plugin developer. Ticket may be closed.


Release a new version to fix the pool hungup bug

Hi,
I use PyPi upgrade Tornado-MySQL to latest stable version（0.5） and I find all connection pool don't work.
If a pool don't set max_open_connections option, the pool won't create new connection and cause program hangup.
This bug already fixed in master branch by this commit, but it don't release a new version after that.
I hope you can release a new version to PyPi as soon as possible.

0.5.1 has been released.


Fix attempt for #1777

Fix #1777
I've tracked down the issue to a task not being properly woken up when notified.
While I haven't had the courage to dig inside of tokio, I know that there have been issues with the multithreaded runner, and therefore this PR switches it out for the single-threaded one.
I'm not sure that this the actual cause. If the test still fails, the issue should be reopened.

test many_nodes_connectivity ... test many_nodes_connectivity has been running for over 60 seconds

Well, that didn't work :man_shrugging: