Can't create or load PGArk map

Hi there,
I'm somehow not able to create or even load procedurally created ark maps with ark server tools.
Things in the configs I changed ->
arkmanager.cfg

Outcommented all lines I didn't need in there (especially serverMap="TheIsland" in this case)

pgark-instance.cfg

arkserverroot="/my/path/blah"
ServerMap="PGARK"
[....]

game.ini

Removed default MapName
PGMapName=ARKproc
PGTerrainPropertiesString=[Generated directly by ARK via local game and copied in here]
[...]

GameUserSettings.ini

serverMap=PGARK   (added this here, too after it didn't work in my instances config)
[...]

Each time I'm trying to start a fresh game with the changes made above, it still creates TheIsland. In none of the config files there is any reference to TheIsland left.
Also, if I'm copying a PGmap I created locally to the server and adapt the save-name etc, the PGmap isn't loaded -- instead a clean TheIsland is created (Edit: or loaded if one exists).
Is there something I've missed or isn't this (otherwise great) tool replacing TheIsland by PGARK in the startup string?
Thanks in advance :)

Oh wow... I just tried it again with serverMap="PGARK" in the arkmanager.cfg aaaand it works. (I wonder what I messed up the first time I've had it that way *scratching my head)
But still: If I'm trying to get the same result by putting it in my instances config or the GameUserSettings it's not working.

pgark-instance.cfg

arkserverroot="/my/path/blah"
ServerMap="PGARK"
[....]

serverMap is case sensitive.

GameUserSettings.ini

serverMap=PGARK (added this here, too after it didn't work in my instances config)
[...]

serverMap is an arkmanager option, not an ARK server option.
Given that the instance file name is pgark-instance.cfg, this would require either using @pgark-instance as the instance specifier, or linking the instance file under an instance name in arkmanager.cfg - e.g. configfile_pgark=/etc/arkmanager/instances/pgark-instance.cfg

Oh, I didn't even realised that I wrote ServerMap instead of serverMap. Writing it the correct way actually works ^^
(And that it's an arkmanager option but not a server option is also good to know, if one is too stupid to get it 👍 )
One more question: Since now I used @instanceName because I only run one server at a time, but set multiple up. How does the linking effect the start up parameter? I guess, if I'm using arkmanager start, all linked configs with the respective servers are going to be loaded? So in my case, linking them in the arkmanager.cfg wouldn't be recommended, right?

When I said "link", it's just specifying what instance config file is used when you specify a particular instance name.  Starting one instance will start only that instance.  Specifying a main instance in arkmanager.cfg simply means that that instance will be acted upon when no instance name is specified.

Aaah, alright. I think I got it. And I'm sorry for those questions, but well... Knowledge is power :'D
Thanks again and have a nice weekend! :)


Method not found Vendr.Extensions.VendrUmbracoBuilderExtensions.AddVendr

Vendr.uSync 2.0.1 breaks the app if using with Vendr 3.0.0.

Removing the package seems to fix the issue and the project can boot up.

Hi @konius There is a new 3.0.0 RC on our unstable feed at https://nuget.outfield.digital/unstable/vendr/v3/index.json if you want to give this a try?

Yes, that seems to have fixed it.


[WFLY-14219] Upgrade module descriptors to 1.9.

Fixes WFLY-14219
Upgrade module descriptor to 1.9:
Exceptions: 1) System property is in use 2) jdeps error.
Please find jdeps results as attachment in JIRA.

@bstansberry I have checked the failed test and it seems to not belong to the PR. Can you check the fail and the PR in total, please?

@boris-unckel I still see non 1.9 module schemes in feature pack modules

I have added some more, additional jdep results are attached to the JIRA https://issues.redhat.com/secure/attachment/12505670/2020-12-16_wfjdep2_22.0.0.Beta1-SNAPSHOT.7z
Remaining which I can't solve
servlet-feature-pack
./ee-8-api/src/main/resources/modules/system/layers/base/javax/transaction/api/main/module.xml:
==> Systempath use
./common/src/main/resources/modules/system/layers/base/org/eclipse/yasson/main/module.xml:
==> jdeps yasson-1.0.5.jar
===> Exception in thread "main" java.lang.module.FindException: Module java.json not found, required by org.eclipse.yasson
ee-feature-pack
./common/src/main/resources/modules/system/layers/base/com/google/guava/main/module.xml:
==> Currently update in WFLY-14142, I want to avoid merge conflicts
./common/src/main/resources/modules/system/layers/base/javax/orb/api/main/module.xml:
==> jdeps --multi-release 11 openjdk-orb-8.1.4.Final.jar
==> Exception in thread "main" java.lang.NullPointerException
at java.base/java.util.stream.ReferencePipeline$3$1.accept(ReferencePipeline.java:195)
./common/src/main/resources/modules/system/layers/base/org/hibernate/5.3/module.xml:
==> slot seems to be in use
ee-feature-pack/pruned/**
==> ignored

@bstansberry @ropalka The tests after a rebase upstream/master are successful. Please review again for merge.

@bstansberry @ropalka The tests after a rebase upstream/master are successful. Please review again for merge.

Automated PR testing doesn't cover WildFly Preview yet but given the significant change this one brings I kicked off a couple custom jobs:
https://ci.wildfly.org/viewLog.html?buildId=238258&tab=buildResultsDiv&buildTypeId=WF_WildflyPreviewNightly
https://ci.wildfly.org/viewLog.html?buildId=238256&tab=buildResultsDiv&buildTypeId=WF_Ee9BootableJarLinuxJdk8

Automated PR testing doesn't cover WildFly Preview yet but given the significant change this one brings I kicked off a couple custom jobs:
https://ci.wildfly.org/viewLog.html?buildId=238258&tab=buildResultsDiv&buildTypeId=WF_WildflyPreviewNightly
https://ci.wildfly.org/viewLog.html?buildId=238256&tab=buildResultsDiv&buildTypeId=WF_Ee9BootableJarLinuxJdk8

@bstansberry Thanks for running the additional test. Indeed there was a missing java.logging module in ee-9...jakarta/xml/bind/api/main/module.xml. I have fixed it. Please review and start the WildFly Preview tests again.

@bstansberry Thanks for running the additional test. Indeed there was a missing java.logging module in ee-9...jakarta/xml/bind/api/main/module.xml. I have fixed it. Please review and start the WildFly Preview tests again.

@boris-unckel Here are the new WildFly Preview runs:
https://ci.wildfly.org/viewLog.html?buildId=238286&tab=buildResultsDiv&buildTypeId=WF_WildflyPreviewNightly
https://ci.wildfly.org/viewLog.html?buildId=238284&tab=buildResultsDiv&buildTypeId=WF_Ee9BootableJarLinuxJdk8

@boris-unckel Here are the new WildFly Preview runs:
https://ci.wildfly.org/viewLog.html?buildId=238286&tab=buildResultsDiv&buildTypeId=WF_WildflyPreviewNightly
https://ci.wildfly.org/viewLog.html?buildId=238284&tab=buildResultsDiv&buildTypeId=WF_Ee9BootableJarLinuxJdk8

@bstansberry Thanks for starting the test runs. Unfortunately we need another one, I have missed one dependency. I have checked all errors, all are caused by this fault. Could you please start a new run?

@bstansberry Thanks for starting the test runs. Unfortunately we need another one, I have missed one dependency. I have checked all errors, all are caused by this fault. Could you please start a new run?

@boris-unckel Here they are:
https://ci.wildfly.org/viewLog.html?buildId=238447&tab=buildResultsDiv&buildTypeId=WF_Ee9BootableJarLinuxJdk8&branch_WF=%2FWFLY-14219_upgrade_module_descriptor_1_8%3B%2Fboris-unckel
https://ci.wildfly.org/viewLog.html?buildId=238446&tab=buildResultsDiv&buildTypeId=WF_WildflyPreviewNightly&branch_WF=%2FWFLY-14219_upgrade_module_descriptor_1_8%3B%2Fboris-unckel

@boris-unckel Here they are:
https://ci.wildfly.org/viewLog.html?buildId=238447&tab=buildResultsDiv&buildTypeId=WF_Ee9BootableJarLinuxJdk8&branch_WF=%2FWFLY-14219_upgrade_module_descriptor_1_8%3B%2Fboris-unckel
https://ci.wildfly.org/viewLog.html?buildId=238446&tab=buildResultsDiv&buildTypeId=WF_WildflyPreviewNightly&branch_WF=%2FWFLY-14219_upgrade_module_descriptor_1_8%3B%2Fboris-unckel

Those looked good.

Very nice, @boris-unckel! 💯

As per the Jakarta EE 8 TCK failure mentioned in https://issues.redhat.com/browse/WFLY-14219?focusedCommentId=15680282&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-15680282, it looks like this change causes TCK failures (in jaxrs, signature test, possibly some others).  I was able to recreate the failure locally with latest wildfly master changes and then reverted this pull request locally and that seemed to help pass the Jakarta EE 8 signature tests.

As per the Jakarta EE 8 TCK failure mentioned in https://issues.redhat.com/browse/WFLY-14219?focusedCommentId=15680282&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-15680282, it looks like this change causes TCK failures (in jaxrs, signature test, possibly some others).  I was able to recreate the failure locally with latest wildfly master changes and then reverted this pull request locally and that seemed to help pass the Jakarta EE 8 signature tests.

@scottmarlow Please give me 45min for a call, after that I'll prepare a fix. The message you provided is very "javax.xml.namespace.QName from [Module "javax.ws.rs.api". I just need a few min.

@scottmarlow Please give me 45min for a call, after that I'll prepare a fix. The message you provided is very "javax.xml.namespace.QName from [Module "javax.ws.rs.api". I just need a few min.

@scottmarlow Please check https://github.com/wildfly/wildfly/pull/13904 for the "javax.xml.namespace.QName from [Module "javax.ws.rs.api" fix

@scottmarlow Please check https://github.com/wildfly/wildfly/pull/13904 for the "javax.xml.namespace.QName from [Module "javax.ws.rs.api" fix


react-native-svg/index has no exported member 'SvgUri'

Question

version: ^9.8.4

Is this a web or native project?

I have the same for RN (0.60.5) with this

Mobile

Is this a web or native project?

mobile

Does that mean react-native or plain mobile web react using dom / react-dom / react-native-web?

Just no definitions for all export elements for xml.js file

Ahaa, would you mind making a pr?

Of course. But not right now. Maybe tomorrow if I dont forget

Added PR
https://github.com/react-native-community/react-native-svg/pull/1092

I've corrected quite a few of the types, and migrated everything internally to typescript now as well.
There's still one use of any left:
https://github.com/react-native-community/react-native-svg/blob/467dd22d3e7132fa30a6f96f15ae341484fc4a97/src/lib/extract/extractResponder.ts#L7-L35
And I'm using unknown in two places:
https://github.com/react-native-community/react-native-svg/blob/467dd22d3e7132fa30a6f96f15ae341484fc4a97/src/elements/Shape.tsx#L6-L26
https://github.com/react-native-community/react-native-svg/blob/467dd22d3e7132fa30a6f96f15ae341484fc4a97/src/lib/util.ts#L1-L12
Any help with removing these / making the types stricter would be greatly appreciated!

Closing as there exists types now, reopen if you find any issues.

@msand I'm having the same issue with Color


User not notified when their draft is saved

Context & versions
https://dev.gov.tools/proposal_discussion
Steps to reproduce

Click on save draft
Nothing happens

Actual behavior

Expected behavior
User is notified that their draft is being saved, and can either safely leave the flow or return to it at a later time.

Please solve this flow so as not to lose the user.

I've changed the Priority to P0 because the user is stuck and they don't know if their draft is saved or not.


Dbaeumer/vivacious-felidae-green

@dbaeumer conflict with main

@bpasero thanks. Can you approve again.


NavArgs toBundle method generate

NavArgs toBundle method code generated
#303

Hey @way-kis
Thanks for this!
It's kind of difficult to parse what is actual change and what is just code formatting. Maybe you could not do auto-formatting here as I'm not really following any auto-formatter since we have lots of special cases in this class.
Besides this, I believe generating a toBundle extension function for each type of NavArgs class will be better to not pollute the DestinationSpec more and also it will lead to better code on caller site:
someNavArgs.toBundle()

Hey @way-kis !
Do you have examples of usage with latest changes? My idea was to really move the extensions out of Destination and DestinationSpec. You can check the NavArgsGetters.kt generated file for a similar thing to what I was thinking about.

example code
`
data class SuvPageActivityNavArgs(
val route: String,
val startDestinationIndex: Int = 0,
val bundle: Bundle = Bundle()
) {
companion object {
fun factory(
graph: NavGraph,
destination: DirectionDestination? = null,
bundle: Bundle = Bundle()
) = SuvPageActivityNavArgs(
route = graph.route,
startDestinationIndex = destination?.let {
graph.destinations.indexOf(it)
} ?: 0,
bundle = bundle
)
}
}
private fun SuvPageActivityNavArgs.findByRouteName(): NavGraph = when (route) {
NavGraphs.sample.route -> {
NavGraphs.sample
}
else -> {
if (BuildConfig.DEBUG) {
error("NotFound graph name!")
}
NavGraphs.root
}
}
@ActivityDestination(navArgsDelegate = SuvPageActivityNavArgs::class)
@AndroidEntryPoint
class SubPageActivity : ComponentActivity() {
private val args by lazy { SubPageActivityDestination.argsFrom(intent) }
override fun onCreate(savedInstanceState: Bundle?) {
super.onCreate(savedInstanceState)
val graph = args.findByRouteName()
intent.putExtras(args.bundle)
val startRoute = when {
args.startDestinationIndex >= 0 -> {
graph.destinations[args.startDestinationIndex]
}
BuildConfig.DEBUG -> {
error("startRute No Include Graph")
}
else -> {
graph.startRoute
}
}
    setContent {
        SampleTheme {
            GraphScreenContainer(
                graph = graph,
                startRoute = startRoute,
                topBar = { navController, typedDestination, navBackStackEntry ->
                    TopBar(
                        navController = navController,
                        destination = typedDestination,
                        navBackStackEntry = navBackStackEntry
                    )
                },
                manualComposableCallsBuilder = {
                    sampleScreen(args)
                }
            )
        }
    }
}

}
fun ManualComposableCallsBuilder.sampleScreen(args: SuvPageActivityNavArgs) {
composable(destination = SampleScreenDestination) {
val argument = SampleScreenDestination.argsFrom(args.bundle)
SampleScreen(key = argument.key)
}
}
@SampleNavGraph(true)
@Destination
@Composable
fun SampleScreen(
key: String = "",
){
}
`

Hey @way-kis !
Do you have examples of usage with latest changes? My idea was to really move the extensions out of Destination and DestinationSpec. You can check the NavArgsGetters.kt generated file for a similar thing to what I was thinking about.

feedback apply. plz check


feat: Remove --templateDir install mode and only use core.hooksPath

The discussion here: https://github.com/gabyx/Githooks/issues/140#issuecomment-1949493549
Should influence how we remove the too complicated templateDir install etc. Githooks should only work with core.hooksPath in the future.

[x] Does it work with Git LFS hooks? -> A test git lfs install installs into the set core.hooksPath, but this will be unset when git hooks uninstall is called, and works flawlessly.
[x] Githooks run wrappers are managed centrally inside the ~/.githooks/hooks install directory. This directory was the former ~/.githooks/templates/hooks which is not needed anymore.
[x] Remove all the boilerplate prompts from the install which are unneccesary.
[x] Keep the registered logic, as it makes updating all registered Githooks easier when updating etc...

Fixed by: #159.


Sprint 2 - Projects - Batch Processing

Batch Processing
Sections
1. Introduction

Introduces the project of building an image processing pipeline that reads image URLs from a CSV file, downloads and processes the images, and uploads them to cloud storage.

2. Learning Objectives

Lists the key learning goals of the project, including defining batch processing, using Go to run existing software, designing a CLI application, and using cloud storage technology.

3. Project Setup

Explains how to set up and run the provided scaffolding code using Docker and Make.

4. Specification

Outlines the requirements for the CLI tool, including reading input CSV, downloading and processing images, uploading to AWS S3, and writing output CSV.

5. How-to

Provides guidance on implementing various aspects of the project, such as reading CSV files, downloading files, image processing, and interacting with AWS S3.

6. Extensions

Suggests additional features and enhancements to the project, like handling failures, avoiding duplicate downloads and uploads, and parallel processing using goroutines.

Key Things To Learn:

Batch processing and how it differs from building servers
Configuring a Makefile to run a Docker image
Using Go to run existing software (ImageMagick) to complete tasks
Designing and building a CLI application to batch process images
Reading from and uploading data to cloud storage technology (AWS S3)
Reading, modifying, and extending existing code
Using the encoding/csv package to read and write CSV files
Downloading files using the http package and handling potential errors and non-200 status codes
Creating and configuring an AWS S3 bucket for storing processed images
Setting up IAM roles and policies to allow access to the S3 bucket
Using the AWS SDK for Go to interact with Amazon S3
Configuring AWS credentials and passing them to the Docker container
Implementing retry-with-backoff strategy for handling temporary failures in downloading and uploading images
Avoiding re-uploading and re-processing the same images without using a database
Using goroutines to process and upload images in parallel for improved performance

https://systems.codeyourfuture.io/projects/batch-processing/
Give me admin access to this repo so I can edit your tickets please

@SallyMcGrath done^


Clippy changes

I made quite a few changes to the codebase, especially the XXX_generator.rs files in order to get everything to pass clippy.

@MrRobotMan @brettfo It would be great to see some or all of this land ...
@MrRobotMan Do you plan to update the PR to restore the .vscode files and resolve the conflicts?

@waywardmonkeys I just restored the original .vscode/settings.json file and merged the conflicts.

@MrRobotMan Cool ... up to @brettfo now then!


[Feature] Update Processing Tasks task approval logic/order

Notion ticket link

Task Management for Processing Tasks

Implementation description

Added some state variables so that steps 4 and on are enabled when previous steps are completed and steps that are dependent on earlier steps are unchecked when the earlier step is undone
Implemented ReviewInformation Modal

Notes

Commented out the form tags in the ReviewInformationModal because when submitted, the page reloads and states reset.

Checklist

[x] My PR name is descriptive, is in imperative tense and starts with one of the following: [Feature],[Improvement] or [Fix],
[x] I have run the appropriate linter(s)
[x] I have requested a review from the RCD team on GitHub, or specific people who are associated with this ticket

@carelynntsai As discussed on Slack, that will be handled in another ticket. Thanks


Default quick pick for selecting a URI doesn't have enough information

Oh I think this is only happening in web. Translation files are different.
Probably just need to update the nls.json file.


500 error on certain add-ons with TB 67.0b2

https://addons.thunderbird.net/en-US/thunderbird/addon/remove-duplicate-messages-alte/ returns a 500 error with TB67, but not via Firefox or TB60.
I suspect an issue with the code to return different versions based on Thunderbird user agent. Not sure why this add-on in particular is an issue though, as other add-ons with no compatible versions are fine.
cc @darktrojan but I will also look at this in the next few days since it's probably hard to reproduce without this specific add-on in your database.

Is there some deleted/disabled version of it? One with no compatibility info? I guess that could cause problems.
Maybe the code I wrote should have a try…except block in an appropriate place…


remote_write: shard up more when backlogged

Fixes #9178
Change the coefficient from 1% to 5%, so instead of targetting to clear the backlog in 100s we target 20s.
Add a unit test which drives the input parameters to calculateDesiredShards() very precisely, to illustrate the behaviour.
Also, round desired shards up before check - previously we would reject an increase from 2 to 2.5 as being
within 30%; by rounding up first we see this as an increase from 2 to 3.
Related: #9236 (although I did not understand that proposal).  I suspect the unit test would be useful for that PR.

my one concern is that by quintupling the speed at which we catch up is there risk of greater oscillations happening?

I tried to create conditions for oscillation in the unit test to check that; if you can suggest more please do.
I think moving the ceil() call will help stop oscillation around low numbers of shards, because comparing the unrounded number meant it would shard up later, by which time the lag has got bigger.
The prohibition on sharding down with >10s lag must slow things down, but it does tend to mean the number we calculate once the lag is gone is at its lowest, hence we are setting things up to change again once lag builds up. Maybe a rule like "don't shard down within 5 minutes of the last change" would be better?
Related: it might be worth expressing the coefficient as "50% per update" (closer to what Callum asked) because I suspect that you always want that to be <100%.
Do we know anyone who is good at control theory?

I agree, I think moving the ceil() call up will help as well. The main scenario I am worried about is if you catch up, then shard down you will automatically be a bit behind as it takes some time to reshard. That could cause sharding up again right away. I also think that would be mostly fixed by https://github.com/prometheus/prometheus/pull/8075.
Generally I am happy with this. Either way on expressing the coefficient as "50% per update" I slightly prefer being able to change the update frequency in your current implementation without having to worry about changing resharding logic, but not a strong preference.

Are you waiting on something from me here?

@cstyan and I were discussing this yesterday, we would like to test this in an internal cluster before merging. Probably easiest to discuss how to do that offline.

Update on 'test in an internal cluster':
We made a branch of Grafana Agent, building off a fork of Prometheus
https://github.com/grafana/agent/blob/dev.improve-rw-sharding/go.mod#L111
with this PR included:
https://github.com/grafana/prometheus/blob/9c127431b579/storage/remote/queue_manager.go#L816
and this has been running in all Grafana Labs clusters since mid-December.
It definitely responds faster to a slowdown, which is the main thing I wanted; possibly too fast because you can get rapid up/down transients like this:

but I think it's good enough to go.
I'll rebase the branch and see if that makes CI happier.

Do you know why in that example the agent sharded down to 5 at all?

I thought that too, but it's an artefact of the way the y-axis is drawn.  It actually went down to 7 - half-way between 5 and 10 on the y-axis.


Stop reporting breadcrumbs to sentry

Sentry's SDK automatically tracks "breadcrumbs", which are pieces of information
that get tracked leading up to an exception. This can be useful for debugging
errors because it gives better insight into what happens before an error occurs,
so you can more easily understand and recreate exactly what happened before an
error occurred.
Unfortunately, Sentry automatically includes all console statements. And since
we use the console a lot (e.g. logging every request received in wrangler dev),
this is mostly useless. Additionally, since developers frequently use the console
to debug their workers we end up with a bunch of data that is not only irrelevant
to the reported error, but also contains data that could be potentially sensitive.
For now, we're turning off breadcrumbs entirely. Later, we might wish to add our
own breadcrumbs manually (e.g. add a "wrangler dev" breadcrumb when a user runs
wrangler dev), at which point we can selectively enable breadcrumbs to catch
only the ones we've put in there ourselves.
I think this closes #516

Is there any way to unit test this?

Looking through Sentry's tests, it looks like they rely on some internals. Looking further into this...

Please have a verbose commit message! I was confused when I saw "no breadcrumbs" in history and had to dive into the commit details and this PR to find out more haha. The PR description here is so nice, it would have made a great commit message too.


Update to new Bluemix Plan names

I had problems deploying to bluemix with the magic button and when I tried to follow the manual steps, I found that some of the plan names had changed in Bluemix.  This branch includes changes for the README which contains the manual steps and the manifest file.
(note that the SSO service is no longer free)

Also fixes #60

Looks good.


go.mod上的名字有问题？

go: github.com/ansible-semaphore/semaphore/api imports
github.com/Sirupsen/logrus<EMAIL_ADDRESS>parsing go.mod:
module declares its path as: github.com/sirupsen/logrus
but was required as: github.com/Sirupsen/logrus

you need to open an issue in the github.com/ansible-semaphore/semaphore repository.
The import path they are using is incorrect.
it should be github.com/sirupsen/logrus (notice the lower-case s)

@chaser24 did you open an issue at ansible-semaphore/semaphore project for them to update the logrus import path case formatting ?


Made Cloudflare DNS optional.

TFC build check is not starting, re-opening PR.

TFC build check is not starting, re-opening PR.


feat: add run_once option for tests

add new retry_once option
add test case for retry_once
make test infrastructure more modular

@theseion Looks like we have conflicts on this one...

Depends on the new scheme changes.

Requires https://github.com/coreruleset/ftw-tests-schema/pull/7

I think this is ready. It currently uses a fixed commit from ftw-tests-schema, so that we can fix the schema, if necessary, before we tag a new version.
This PR has grown quite a bit because many refactorings came in handy and I opted to adjust the schema on-the-fly where I thought it made sense.
There's a draft PR for CRS where I've made all the necessary changes to support the code here: https://github.com/coreruleset/coreruleset/pull/3369. With that PR, all tests for both nginx and httpd succeed (including all previously disabled and commented tests).
Note that Coraza support will come a bit later with https://github.com/coreruleset/coreruleset/pull/3377.
It would be great if I could also get feedback from @jcchavezs and @christiantreutler-avi on how these changes will affect you.
@jcchavezs Note that I opted to drop the replace for koanf/maps because it is annoying. If that's a problem for Coraza, let me know.

Well, honestly, is too big to understand cross implications between things. I'll approve and 🤞


fix the ref for the GitHub badge while on the root page

The Hugo script for the GitHub badge was returning an empty page
when on the root of the site, which then got concatenated into
a URL that didn't exist on GitHub. This commit checks to see if
that page portion is empty, and if so sets the URL for the badge
to the OpenTelemetry organization page

I signed the CLA.

Thanks! This is probably a better option than how I had it before.


[C#] feat: add OpenAIModel unit test

Linked issues
closes: #1115
Details
Provide a list of your changes here. If you are fixing a bug, please provide steps to reproduce the bug.
Change details

Describe your changes, with screenshots and code snippets as appropriate

code snippets:
screenshots:
Attestation Checklist

[ ] My code follows the style guidelines of this project

I have checked for/fixed spelling, linting, and other errors

I have commented my code for clarity

I have made corresponding changes to the documentation (we use TypeDoc to document our code)

My changes generate no new warnings

I have added tests that validates my changes, and provides sufficient test coverage. I have tested with:

Local testing
E2E testing in Teams

New and existing unit tests pass locally with my changes

Additional information

Feel free to add other relevant information below


Installing composer and php in the docker container

Hi,
I've been playing around with this package and I am highly impressed!  I'm trying to use this to work on my Laravel application and a problem I've run across is not having access to composer or PHP in the docker container this is running on.  How would I go about installing that?
Another issue I'm having is sometimes I run into an issue where it's unable to edit files correctly.  It ends up appending the new data to the end of the file, then tries correcting itself and gets into a loop doing that.  I assume it must be some sort of permissions issue but it's weird because it doesn't happen all the time.  Do you have any solutions for this?
Thank you very much for this awesome package!  I appreciate your help :)

Hello there and thanks for trying OpenHands!
Installing php. You can try two things:

You can ask it to install php and see how it goes. It should be able to install it but if it takes longer than 2 minutes, you might get a timeout and will have to modify the timeout. The problem with this method is if OpenHands restarts or crashes for some reason, you have to install it again.
The other way is to create a docker container that has all the packages you need via this method: https://docs.all-hands.dev/modules/usage/how-to/custom-sandbox-guide. Then just use that docker container and you won't need to reinstall anything, even if OpenHands restarts.

For your editing issue, which model are you using? Are you using gpt-4o? There is an open editing issue with gpt-4o currently: https://github.com/All-Hands-AI/OpenHands/issues/3679
There is work being done on improving the editing of files so hopefully this gets fixed in the near future.

Hi Mamoodi!  I have been mostly using claude sonnet 3.5 since it has the context caching working.  I can't wait for Gemini to work because my project is using up all of Claude's context.
I've found that it struggles to edit files consistently.  I'm not sure if it's a permissions issue but it's very unpredictable.  I'm not a very advanced programmer so it's hard for me to debug it but what I seem to find is that after a long conversation it starts not being able to edit or overwrite files.  What it's trying to insert ends up at the end of the file creating a duplicate.  The only solution I've found so far is to go into docker and delete the image and then rebuild the whole thing and then it works again for a while.  My only guess since it happens mid conversation is that it might have to do with the prompt caching and possibly once the ttl of the cache has expired and the context refreshes then the ability to edit files is disrupted.  That's just a guess though and I haven't dug into your code to see how any of this works.
Another thing that's caused me a lot of issues is when the model seems to go rogue.  I ask it to do something and then it starts doing something completely different.  It's almost like it reads some instructions in a file and starts going on it's own path and the pause button doesn't seem to stop it in time.    I wonder if there's a way to group a set of actions and then approve the set and then have it do the set as opposed to every single thing it does.  Also not being able to toggle the setting to turn confirmation mode on and off during a conversation is not ideal.
Another bug I've found is that when I write a response in the middle of an action during confirmation mode, the whole thing stalls and the next action doesn't have an option to approve it or deny it.  I have to reload the page and resume the session to get it to work again.
I appologize I'm not able to help with the codebase but I hope I can be helpful in finding bugs that become evident while using the system.
Thank you!
Amir

Hi Amir, thanks for your detailed write up.
Regarding the editing issue, there is definitely improvements that can be done on the OpenHands side. I definitely know there is work being done but not sure when it will be ready. Here is the mega issue that is relevant to editing: https://github.com/All-Hands-AI/OpenHands/issues/3231
The going rogue issue. I have run into this myself where I give it something vague and it starts going on a completely different path. For this, the solution has been to give it more detailed instructions. For example I might say "Create this file that does X but do not run it" or "You already have a project setup in your workspace....". Otherwise it finds the files and starts creating an app haha.
My number one suggestion right now is when you give it a prompt and it doesn't behave correctly, press the Thumbs Down button on the UI and share it if you are able to. This will give feedback to the devs to be able to make things better:
https://docs.all-hands.dev/modules/usage/feedback
Hopefully OpenHands can get better on these issues over time.

Hi Mamoodi,
Quick question, I've noticed that the best way to avoid the errors during the editing of files is if the file is deleted and replaced.  I have to ask it each time to do that but then the next time it tries to edit again.  Is there a way to give it instructions that it will retain in it's context?

That's a good question. I'm not an expert in how LLMs behave but you could try giving it a prompt like:
"From now on, whenever you need to edit a file, just replace the file entirely with the new content". See how that goes. This might work if your file has little content. Unfortunate this happens though. Hopefully this gets fixed soon.

Hi @mamoodi I was curious if you know of a way to give custom instructions to the model that will persist longer than a few turns?  I find that it keep repeating the same mistakes and it would be nice to have some instructions that stick with it like you can in the ui of claude and chatgpt.

OpenHands doesn't specifically provide that functionality. If you tell the LLM to "Never do X from now on" and it does do it... then maybe you can open a bug specifying your instructions and logs of when it goes against your ask.
I don't know if this would be a OpenHands bug or what but if you open one with more information, at least someone can take a look.


GZIP compression does not work for responses to POST requests.

Wiremock registers GzipFilter with default configuration:
com.github.tomakehurst.wiremock.jetty9.JettyHttpServer

mockServiceContext.addFilter(GzipFilter.class, "/*", EnumSet.of(DispatcherType.REQUEST, DispatcherType.FORWARD));

But by default the aforementioned filter handles only GET request.
org.eclipse.jetty.servlets.GzipFilter

// public void init
tmp=filterConfig.getInitParameter("methods");
        if (tmp!=null)
        {
            StringTokenizer tok = new StringTokenizer(tmp,",",false);
            while (tok.hasMoreTokens())
                _methods.add(tok.nextToken().trim().toUpperCase(Locale.ENGLISH));
        }
        else
            _methods.add(HttpMethod.GET.asString());

// public void doFilter()
if (!_methods.contains(request.getMethod()) || isExcludedPath(requestURI))
        {
            super.doFilter(request,response,chain);
            return;
        }

Has this been updated yet?

WireMock 2.18.0 switched to the new Jetty GZip handler. I haven't specifically tested it for this case yet, but it's worth giving it a try.

Thanks,
I have never used gzip data in wiremock before but have been asked to start.
Does wiremock accept gzip data during the seeding process via the endpoints or does it take uncompressed data and gzips it before sending it if the Accept-Encoding is gzip?
From: Tom Akehurst<EMAIL_ADDRESS>Sent: Thursday, June 14, 2018 9:00 AM
To: tomakehurst/wiremock<EMAIL_ADDRESS>Cc: Jay Goldberg<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [tomakehurst/wiremock] GZIP compression does not work for responses to POST requests. (#787)
WireMock 2.18.0 switched to the new Jetty GZip handler. I haven't specifically tested it for this case yet, but it's worth giving it a try.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Ftomakehurst%2Fwiremock%2Fissues%2F787%23issuecomment-397306031&data=02|01|jgoldberg%40pros.com|456e56c2cc724103abdc08d5d1ff21e8|094cfb7ad13146379047e339e7d04359|0|0|636645816078219778&sdata=DgT%2FwpHUDmtD3cmoLbV24r65vIGM%2Bayckab%2FE1oSD6E%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAD0hEOLWXUL6dVjEFnFXPg8LehXbaFHLks5t8mxlgaJpZM4QEJHW&data=02|01|jgoldberg%40pros.com|456e56c2cc724103abdc08d5d1ff21e8|094cfb7ad13146379047e339e7d04359|0|0|636645816078219778&sdata=wdzkLlEfO1cbZlwLfb2lYQWBzTO9lAWLBS1LufCEqYs%3D&reserved=0.

Stubs are always loaded uncompressed, then WireMock (well, Jetty's handler specifically) gzips/gunzips requests and responses.

Hi Tom,
Regarding the change in 2.18.0 for the new Jetty Gzip Handler, does it mean that I can change the Jetty version to 9.4 when I build wiremock locally? I'm using the standalone jar and the Jetty version incuded seems 9.2.x , so I'm not sure if there's something else I need to allow "POST" methods to apply Gzip Handler.. Thanks!

I just bumped into this scenario with v<IP_ADDRESS>. Do we have updates  on this feature request? Thanks.

@jecklgamis , i did checked-out the wiremock code and build it locally, to enable gzip-compression on HTTP POST requests.. @tomakehurst , could we have it configurable when running wiremock standalone on the next release? Thanks!

What was the change you had to make?
Are we talking about compressing the request or response body?

Did not make a change. I don’t think I got it to work. It has been awhile. I was trying to compress the response from wiremock to my client so that the client can test its un-compress method.
From: Tom Akehurst<EMAIL_ADDRESS>Sent: Thursday, August 9, 2018 9:56 AM
To: tomakehurst/wiremock<EMAIL_ADDRESS>Cc: Jay Goldberg<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [tomakehurst/wiremock] GZIP compression does not work for responses to POST requests. (#787)
What was the change you had to make?
Are we talking about compressing the request or response body?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Ftomakehurst%2Fwiremock%2Fissues%2F787%23issuecomment-411787132&data=02|01|jgoldberg%40pros.com|f553d3b8c4bd467d2d5808d5fe084ac0|094cfb7ad13146379047e339e7d04359|0|0|636694233925221921&sdata=AgRD1VETFSyI0ThjLDmqr1K%2BowVbECr4vc3XAvGac%2BM%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAD0hEBeaabwNIk5Mw9hKi0HV6SIa6XLpks5uPE2dgaJpZM4QEJHW&data=02|01|jgoldberg%40pros.com|f553d3b8c4bd467d2d5808d5fe084ac0|094cfb7ad13146379047e339e7d04359|0|0|636694233925231926&sdata=DugOHSKuTm6lLVKWYRp4o0pkspHDhleoYhwrQ95zZaA%3D&reserved=0.

OK, I've found the problem. I'll include some kind of fix in the next release.

Nice!
From: Tom Akehurst<EMAIL_ADDRESS>Sent: Thursday, August 9, 2018 10:32 AM
To: tomakehurst/wiremock<EMAIL_ADDRESS>Cc: Jay Goldberg<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [tomakehurst/wiremock] GZIP compression does not work for responses to POST requests. (#787)
OK, I've found the problem. I'll include some kind of fix in the next release.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Ftomakehurst%2Fwiremock%2Fissues%2F787%23issuecomment-411799553&data=02|01|jgoldberg%40pros.com|31f68d16a2194af6545108d5fe0d5048|094cfb7ad13146379047e339e7d04359|0|0|636694255492170379&sdata=4G%2FzDRX%2FbJUgcQtq7sds1SfOaQMYAXH%2Fm16X8o8DZ6k%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAD0hEMwKEwMsk8ArL9wsQwo9tKtvBWLuks5uPFYKgaJpZM4QEJHW&data=02|01|jgoldberg%40pros.com|31f68d16a2194af6545108d5fe0d5048|094cfb7ad13146379047e339e7d04359|0|0|636694255492180388&sdata=lkNCnClMP0BQ%2FBaeZ4bw5%2FXgMNVtOwVcH2kn32hzP8E%3D&reserved=0.

I updated to 2.18.0 today, and noticed calling WireMock.reset() sent an empty POST to __admin/reset , which in turn caused an EOFException when trying to unzip the request body.

Can you provide some more details so I can replicate?

Using libraries:
jetty-server:9.4.11.v20180605
wiremock:2.18.0
        WireMockServer wireMockServer = new WireMockServer(STUB_TEST_PORT);         WireMock.configureFor("localhost", STUB_TEST_PORT);         wireMockServer.start();         WireMock.reset();
Stack trace:
java.lang.RuntimeException: java.io.EOFException 	at com.github.tomakehurst.wiremock.servlet.WireMockHttpServletRequestAdapter.getBody(WireMockHttpServletRequestAdapter.java:137) 	at com.github.tomakehurst.wiremock.servlet.WireMockHttpServletRequestAdapter.getBodyAsString(WireMockHttpServletRequestAdapter.java:159) 	at com.github.tomakehurst.wiremock.http.AdminRequestHandler.handleRequest(AdminRequestHandler.java:67) 	at com.github.tomakehurst.wiremock.http.AbstractRequestHandler.handle(AbstractRequestHandler.java:44) 	at com.github.tomakehurst.wiremock.servlet.WireMockHandlerDispatchingServlet.service(WireMockHandlerDispatchingServlet.java:108) 	at javax.servlet.http.HttpServlet.service(HttpServlet.java:790) 	at org.eclipse.jetty.servlet.ServletHolder.handle(ServletHolder.java:865) 	at org.eclipse.jetty.servlet.ServletHandler.doHandle(ServletHandler.java:535) 	at org.eclipse.jetty.server.handler.ScopedHandler.nextHandle(ScopedHandler.java:255) 	at org.eclipse.jetty.server.handler.ContextHandler.doHandle(ContextHandler.java:1317) 	at org.eclipse.jetty.server.handler.ScopedHandler.nextScope(ScopedHandler.java:203) 	at org.eclipse.jetty.servlet.ServletHandler.doScope(ServletHandler.java:473) 	at org.eclipse.jetty.server.handler.ScopedHandler.nextScope(ScopedHandler.java:201) 	at org.eclipse.jetty.server.handler.ContextHandler.doScope(ContextHandler.java:1219) 	at org.eclipse.jetty.server.handler.ScopedHandler.handle(ScopedHandler.java:144) 	at org.eclipse.jetty.server.Dispatcher.forward(Dispatcher.java:203) 	at org.eclipse.jetty.server.Dispatcher.error(Dispatcher.java:81) 	at com.github.tomakehurst.wiremock.jetty9.NotFoundHandler.handle(NotFoundHandler.java:41) 	at org.eclipse.jetty.server.Response.sendError(Response.java:655) 	at org.eclipse.jetty.server.Response.sendError(Response.java:590) 	at com.github.tomakehurst.wiremock.servlet.WireMockHandlerDispatchingServlet$ServletHttpResponder.respondTo(WireMockHandlerDispatchingServlet.java:175) 	at com.github.tomakehurst.wiremock.servlet.WireMockHandlerDispatchingServlet$ServletHttpResponder.respondSync(WireMockHandlerDispatchingServlet.java:138) 	at com.github.tomakehurst.wiremock.servlet.WireMockHandlerDispatchingServlet$ServletHttpResponder.respond(WireMockHandlerDispatchingServlet.java:132) 	at com.github.tomakehurst.wiremock.http.AbstractRequestHandler.handle(AbstractRequestHandler.java:63) 	at com.github.tomakehurst.wiremock.servlet.WireMockHandlerDispatchingServlet.service(WireMockHandlerDispatchingServlet.java:108) 	at javax.servlet.http.HttpServlet.service(HttpServlet.java:790) 	at org.eclipse.jetty.servlet.ServletHolder.handle(ServletHolder.java:865) 	at org.eclipse.jetty.servlet.ServletHandler.doHandle(ServletHandler.java:535) 	at org.eclipse.jetty.server.handler.ScopedHandler.nextHandle(ScopedHandler.java:255) 	at org.eclipse.jetty.server.handler.ContextHandler.doHandle(ContextHandler.java:1317) 	at org.eclipse.jetty.server.handler.ScopedHandler.nextScope(ScopedHandler.java:203) 	at org.eclipse.jetty.servlet.ServletHandler.doScope(ServletHandler.java:473) 	at org.eclipse.jetty.server.handler.ScopedHandler.nextScope(ScopedHandler.java:201) 	at org.eclipse.jetty.server.handler.ContextHandler.doScope(ContextHandler.java:1219) 	at org.eclipse.jetty.server.handler.ScopedHandler.handle(ScopedHandler.java:144) 	at org.eclipse.jetty.server.handler.gzip.GzipHandler.handle(GzipHandler.java:674) 	at org.eclipse.jetty.server.handler.HandlerCollection.handle(HandlerCollection.java:126) 	at org.eclipse.jetty.server.handler.HandlerWrapper.handle(HandlerWrapper.java:132) 	at org.eclipse.jetty.server.Server.handle(Server.java:531) 	at org.eclipse.jetty.server.HttpChannel.handle(HttpChannel.java:352) 	at org.eclipse.jetty.server.HttpConnection.onFillable(HttpConnection.java:260) 	at org.eclipse.jetty.io.AbstractConnection$ReadCallback.succeeded(AbstractConnection.java:281) 	at org.eclipse.jetty.io.FillInterest.fillable(FillInterest.java:102) 	at org.eclipse.jetty.io.ChannelEndPoint$2.run(ChannelEndPoint.java:118) 	at org.eclipse.jetty.util.thread.QueuedThreadPool.runJob(QueuedThreadPool.java:762) 	at org.eclipse.jetty.util.thread.QueuedThreadPool$2.run(QueuedThreadPool.java:680) 	at java.lang.Thread.run(Thread.java:748) Caused by:  java.io.EOFException 	at java.util.zip.GZIPInputStream.readUByte(GZIPInputStream.java:268) 	at java.util.zip.GZIPInputStream.readUShort(GZIPInputStream.java:258) 	at java.util.zip.GZIPInputStream.readHeader(GZIPInputStream.java:164) 	at java.util.zip.GZIPInputStream.<init>(GZIPInputStream.java:79) 	at java.util.zip.GZIPInputStream.<init>(GZIPInputStream.java:91) 	at com.github.tomakehurst.wiremock.common.Gzip.unGzip(Gzip.java:36) 	at com.github.tomakehurst.wiremock.servlet.WireMockHttpServletRequestAdapter.getBody(WireMockHttpServletRequestAdapter.java:135) 	at com.github.tomakehurst.wiremock.servlet.WireMockHttpServletRequestAdapter.getBodyAsString(WireMockHttpServletRequestAdapter.java:159) 	at com.github.tomakehurst.wiremock.http.AdminRequestHandler.handleRequest(AdminRequestHandler.java:67) 	at com.github.tomakehurst.wiremock.http.AbstractRequestHandler.handle(AbstractRequestHandler.java:44) 	at com.github.tomakehurst.wiremock.servlet.WireMockHandlerDispatchingServlet.service(WireMockHandlerDispatchingServlet.java:108) 	at javax.servlet.http.HttpServlet.service(HttpServlet.java:790) 	at org.eclipse.jetty.servlet.ServletHolder.handle(ServletHolder.java:865) 	at org.eclipse.jetty.servlet.ServletHandler.doHandle(ServletHandler.java:535) 	at org.eclipse.jetty.server.handler.ScopedHandler.nextHandle(ScopedHandler.java:255) 	at org.eclipse.jetty.server.handler.ContextHandler.doHandle(ContextHandler.java:1317) 	at org.eclipse.jetty.server.handler.ScopedHandler.nextScope(ScopedHandler.java:203) 	at org.eclipse.jetty.servlet.ServletHandler.doScope(ServletHandler.java:473) 	at org.eclipse.jetty.server.handler.ScopedHandler.nextScope(ScopedHandler.java:201) 	at org.eclipse.jetty.server.handler.ContextHandler.doScope(ContextHandler.java:1219) 	at org.eclipse.jetty.server.handler.ScopedHandler.handle(ScopedHandler.java:144) 	at org.eclipse.jetty.server.Dispatcher.forward(Dispatcher.java:203) 	at org.eclipse.jetty.server.Dispatcher.error(Dispatcher.java:81) 	at com.github.tomakehurst.wiremock.jetty9.NotFoundHandler.handle(NotFoundHandler.java:41) 	at org.eclipse.jetty.server.Response.sendError(Response.java:655) 	at org.eclipse.jetty.server.Response.sendError(Response.java:590) 	at com.github.tomakehurst.wiremock.servlet.WireMockHandlerDispatchingServlet$ServletHttpResponder.respondTo(WireMockHandlerDispatchingServlet.java:175) 	at com.github.tomakehurst.wiremock.servlet.WireMockHandlerDispatchingServlet$ServletHttpResponder.respondSync(WireMockHandlerDispatchingServlet.java:138) 	at com.github.tomakehurst.wiremock.servlet.WireMockHandlerDispatchingServlet$ServletHttpResponder.respond(WireMockHandlerDispatchingServlet.java:132) 	at com.github.tomakehurst.wiremock.http.AbstractRequestHandler.handle(AbstractRequestHandler.java:63) 	at com.github.tomakehurst.wiremock.servlet.WireMockHandlerDispatchingServlet.service(WireMockHandlerDispatchingServlet.java:108) 	at javax.servlet.http.HttpServlet.service(HttpServlet.java:790) 	at org.eclipse.jetty.servlet.ServletHolder.handle(ServletHolder.java:865) 	at org.eclipse.jetty.servlet.ServletHandler.doHandle(ServletHandler.java:535) 	at org.eclipse.jetty.server.handler.ScopedHandler.nextHandle(ScopedHandler.java:255) 	at org.eclipse.jetty.server.handler.ContextHandler.doHandle(ContextHandler.java:1317) 	at org.eclipse.jetty.server.handler.ScopedHandler.nextScope(ScopedHandler.java:203) 	at org.eclipse.jetty.servlet.ServletHandler.doScope(ServletHandler.java:473) 	at org.eclipse.jetty.server.handler.ScopedHandler.nextScope(ScopedHandler.java:201) 	at org.eclipse.jetty.server.handler.ContextHandler.doScope(ContextHandler.java:1219) 	at org.eclipse.jetty.server.handler.ScopedHandler.handle(ScopedHandler.java:144) 	at org.eclipse.jetty.server.handler.gzip.GzipHandler.handle(GzipHandler.java:674) 	at org.eclipse.jetty.server.handler.HandlerCollection.handle(HandlerCollection.java:126) 	at org.eclipse.jetty.server.handler.HandlerWrapper.handle(HandlerWrapper.java:132) 	at org.eclipse.jetty.server.Server.handle(Server.java:531) 	at org.eclipse.jetty.server.HttpChannel.handle(HttpChannel.java:352) 	at org.eclipse.jetty.server.HttpConnection.onFillable(HttpConnection.java:260) 	at org.eclipse.jetty.io.AbstractConnection$ReadCallback.succeeded(AbstractConnection.java:281) 	at org.eclipse.jetty.io.FillInterest.fillable(FillInterest.java:102) 	at org.eclipse.jetty.io.ChannelEndPoint$2.run(ChannelEndPoint.java:118) 	at org.eclipse.jetty.util.thread.QueuedThreadPool.runJob(QueuedThreadPool.java:762) 	at org.eclipse.jetty.util.thread.QueuedThreadPool$2.run(QueuedThreadPool.java:680) 	at java.lang.Thread.run(Thread.java:748)

@tomakehurst this is the code I've modified to enable POST request to use compression when the request has "Accept-Encoding:gzip" .. I applied this to use it on running wiremock as standalone jar.
on class JettyHttpServer:addGzipHandler(...)
gzipWrapper.addIncludedMethods(HttpMethod.POST.asString());

Thanks. I've actually got a patch for this on in my workspace now so it'll be on master shortly.

Thanks @tomakehurst, @joshuacura . I have a similar change in my local. Looks like Dropwizard app has this a similar behavior and it has to be explicitly configured  (https://www.dropwizard.io/1.3.5/docs/manual/configuration.html).

Hey @tomakehurst! Did you manage to push the fix to master for this?
I am making a POST request with "Content-Encoding: gzip" header and I get a java.util.zip.ZipException: Not in GZIP format. If I remove this header, I get the response as expected. Is there a way to fix this? I am using WireMock as standalone via JSON.
@joshuacura Where did you apply this fix?

@stelarelas yes, I should have closed this issue ages ago, as it's been fixed.
It sounds to me like you're trying to send a non-gzipped request with a gzip header, so I'd expect it to throw an exception in that instance.

Fixed in df78bfca

@tomakehurst Thanks for the quick response! The situation is exactly that. My production endpoint returns a gzip response that's why there is a "Content-Encoding: gzip" header in it. How am I suppose to mock this response in my json file that WireMock uses? I currently have it as a json and I get the Not in GZIP format error back.

It should happen automatically provided you're sending Accept-Encoding:gzip in the request.

@tomakehurst - I am also facing the same problem and trying to find a solution on this.

Using Wiremock standalone version.
My production URL returns the response with headers-   "Content-Encoding":[ "deflate,gzip"]
I mocked this request with RR pairs but ending up getting -HTTP 500 error

java.lang.RuntimeException: java.io.EOFException
4. how should I build the request so that it will accept the request and respond.
5. I tried following writing in my request body but didn't solve the problem-
{
  "id" : "3ee520c4-905b-4be2-97d8-1ec69ea992d9",
  "name" : "rm_shopping_v2_flights",
  "request" : {
    "url" : "/rm/shopping/v2/flights",
    "method" : "POST",
	"headers" : {
      "Accept-Encoding" : {
	  "contains" : "deflate,gzip"
	  }
	}
      }
6. I tried removing headers from the request being sent to mock, it works but with header "Content-Encoding":[ "deflate,gzip"], it returning in 500 error.
7.Kindly help. This is an urgent issue and we have many apis like that to work with. Appreciate your help.


Transitive uplink dependencies provide a 404 error

Your Environment

verdaccio version: 5.26.3
node version [12.x.x, 14.x.x]: v18.18.0
package manager: npm@9
os: linux
platform: GitHub Actions (Docker)

Describe the bug
When run in GitHub Actions, Verdaccio presents a 404 error for a transitive dependency (in this case, @types/yargs-parser) that should be passed through from an npm uplink proxy.
One notable fact is that I believe this is a library that has multiple versions in the dependency graph, so this may be an example of the same bug as #3530.
To Reproduce
Example repository
See .github/workflows/ci.yml to see what this example does in GitHub Actions. In brief:

The proxy is run with the proxy/proxy.sh script using the proxy/config.yml config file.
Then a dummy user is authenticated in order to publish packages to Verdaccio.
Next the super-duper-library package is published to the Verdaccio proxy.
Finally, the test/smoke-test package is built, using the Verdaccio proxy.

Expected behavior
The above steps should succeed because the Verdaccio proxy should be serving the one super-duper-library package and all other dependencies should be passed through successfully from the npm uplink.
Screenshots, server logs, package manager log

Complete GitHub Actions log
Configuration File
storage: ./storage
auth:
  htpasswd:
    file: ./htpasswd
uplinks:
  npmjs:
    url: https://registry.npmjs.org/
    timeout: 120s
packages:
  'super-duper-library':
    access: $all
    publish: $all
  '**':
    access: $all
    publish: $authenticated
    proxy: npmjs
log: { type: file, path: proxy.log, level: info }

Environment information
Environment Info:
  System:
    OS: Linux 6.2 Ubuntu 22.04.3 LTS 22.04.3 LTS (Jammy Jellyfish)
    CPU: (2) x64 Intel(R) Xeon(R) CPU E5-2673 v3 @ 2.40GHz
  Binaries:
    npm: 9.8.1 - /opt/hostedtoolcache/node/18.18.0/x64/bin/npm
  Virtualization:
    Docker: 24.0.6 - /usr/bin/docker
  Browsers:
    Chrome: 117.0.5938.132

Contribute to Verdaccio

[ ] I'm willing to fix this bug 🥇

I noticed that this particular error message in the above screenshot looks a bit suspicious:
npm ERR! 404  '@types/yargs-parser<EMAIL_ADDRESS>is not in this registry.

The @types/yargs-parser@http://etc part seems fishy, doesn't it? Could this maybe suggest there's a bug in uplinks having something to do with the @ or / characters?

Let me also try this with the latest pre-release of verdaccio 6 and see if that does anything.

I also have this situation. After setting uplinks, when downloading non-existent packages from verdaccio, the download and cache from uplinks will not be cached in verdaccio.

The test above does not work anymore. I forked and updated it.
https://github.com/mbtools/verdaccio-proxy-config-test-case
The issue is reproducible using next-7 (see the repo). The 404 error occurs only when trying to get the tarball of scoped packages.
I was not able to reproduce the issue on my machine although I followed the same steps. Hmmm.
Hopefully, https://github.com/verdaccio/verdaccio/pull/4847 will provide more insights.

fixed at https://github.com/verdaccio/verdaccio/releases/tag/v6.0.2

That's amazing! Thanks so much for keeping on top of this, @juanpicado. I made a Verdaccio sponsorship donation to show my appreciation for this great tool.


Deposit simulation fails after Eth2Genesis during PoW simulation from a JSON file

When running deposit simulation in mode = "simulation" with input file inputFile = "validator_test_data.json" the following java.lang.ArithmeticException: / by zero error is thrown when in method get_beacon_proposer_index.
In our current implementation v0.7.1 debugging is difficult (4 min wait time). We are waiting until after changes are made to how our DepositContract is deployed to begin debugging.
> Task :run
Setting logging level to INFO
10:40:58.948 [INFO ] - ******* Eth2Genesis Event detected ******* : 0xD56342AFD67EC47A07545D73D78F1A545D6E294BE7A4BF3D34BF23319355CC80
10:40:58.948 [INFO ] - Node slot: 0
10:40:58.948 [INFO ] - Node time: 0
10:41:00.022 [INFO ] - Initial state root is 0xAA48CB73258733B18CCC90E565074C3D3B8958312CD4346EA441D3CF2BCB8588
10:41:00.022 [INFO ] - Genesis block root is 0x010E77C10B6AD09B59ED1D47A8B6DEC78F93C7B342FE10FEA0BD41B23A7A6ADD
Jul 10, 2019 10:41:00 AM com.google.common.eventbus.EventBus$LoggingHandler handleException
SEVERE: Exception thrown by subscriber method onGenesisHeadStateEvent(tech.pegasys.artemis.statetransition.GenesisHeadStateEvent) on subscriber tech.pegasys.artemis.validator.coordinator.ValidatorCoordinator@3ffb3598 when dispatching event: tech.pegasys.artemis.statetransition.GenesisHeadStateEvent@1f137bcd
java.lang.ArithmeticException: / by zero
	at com.google.common.primitives.UnsignedLongs.remainder(UnsignedLongs.java:294)
	at com.google.common.primitives.UnsignedLong.mod(UnsignedLong.java:171)
	at tech.pegasys.artemis.datastructures.util.BeaconStateUtil.get_beacon_proposer_index(BeaconStateUtil.java:592)
	at tech.pegasys.artemis.validator.coordinator.ValidatorCoordinator.createBlockIfNecessary(ValidatorCoordinator.java:403)
	at tech.pegasys.artemis.validator.coordinator.ValidatorCoordinator.onNewHeadStateEvent(ValidatorCoordinator.java:258)
	at tech.pegasys.artemis.validator.coordinator.ValidatorCoordinator.onGenesisHeadStateEvent(ValidatorCoordinator.java:187)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.base/java.lang.reflect.Method.invoke(Method.java:566)
	at com.google.common.eventbus.Subscriber.invokeSubscriberMethod(Subscriber.java:87)
	at com.google.common.eventbus.Subscriber$SynchronizedSubscriber.invokeSubscriberMethod(Subscriber.java:144)
	at com.google.common.eventbus.Subscriber$1.run(Subscriber.java:72)
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
	at java.base/java.lang.Thread.run(Thread.java:834)

> Task :run FAILED

config.toml
[deposit]
# normal, test, simulation
# "test" pre-production
# "simulation" to run a simulation of deposits with ganache-cli, if a inputFile is included the file will replay the deposits
# "normal" production, must include contractAddr and nodeUrl
mode = "simulation"
inputFile = "validator_test_data.json"
numValidators = 16
numNodes = 1
contractAddr = "0x77f7bED277449F51505a4C54550B074030d989bC"
nodeUrl = "http://localhost:7545"

@cemozerr debugged this and found the issue was that the was the bls_verify method in BlockProcessorUtil.process_deposits that is failing to validate the BLSSignature from the validator_test_data.json.
Any fix should wait until issue #771 is resolved that will allow a more robust tool set to make a fix.


add Executor::run_with_local

Alternative to #3
This adds a run_with_local method to Executor, which creates a LocalExecutor and runs it alongside the main one, to allow !Send futures to be spawned.

Thanks for the PR! But, I think this change is not applicable anymore because we don't have Task::local() anymore, so I'm going to close. Sorry! I feel like I'm closing so many PRs that other people have put effort into. :(


feat/149-frontend

Created Dialog modal component and added it to the delete button of the Manage Access screen.

Hey Nick, I'm merging your branch into mine locally so I can control the merge, the PR can be closed (not merged).


Updating gradient regurarly increases amount of node elements and overloads the page

Describe the bug
I have a usecase when i need to update the gradient on the svg path multiple times.
Unfortunately every time I call the render method, it doesn't override the previous gradient node, but it creates new ones.
This becomes a problem after some time, when I end up 5k node elements, which is very bad for performance.
To Reproduce
Steps to reproduce the behavior:

Add gradient to an svg path
use .render method to render gradients
do it multiple times
check performance monitor in chrome and you'll see that the node count is going up

Expected behavior
I'd expect to have a cleanup process before the gradient renders

Did you find any way to fix this?

unfortunately not yet


Cluster is added behind advanced marker which near to cluster location

Thanks for stopping by to let us know something could be better!

PLEASE READ
If you have a support contract with Google, please create an issue in the support console. This will ensure a timely response.
Discover additional support services for the Google Maps Platform, including developer communities, technical guidance, and expert support at the Google Maps Platform support resources page.
If your bug or feature request is not related to this particular library, please visit the Google Maps Platform issue trackers.
Check for answers on StackOverflow with the google-maps tag.

Please be sure to include as much information as possible:
Environment details

Specify the API at the beginning of the title (for example, "Places: ...")
OS type and version
Library version and other environment information

Steps to reproduce

Create a cluster and customize it  by renderer option
Add few markers
Add an advanced marker near to cluster location
Make cluster appear on top of advanced marker

Code example
# example
https://jsfiddle.net/dt4fjp2L/

#### Stack trace

example

Following these steps will guarantee the quickest resolution possible.

Thanks!

If you would like to upvote the priority of this issue, please comment below or react on the original post above with :+1: so we can see what is popular when we triage.
@aneela1191 Thank you for opening this issue. 🙏
Please check out these other resources that might help you get to a resolution in the meantime:

Check the issue tracker - bugs and feature requests for Google Maps Platform APIs and SDKs
Open a support case - Get 1:1 support in Cloud Console.
Discord - chat with other developers
StackOverflow - use the google-maps tag

This is an automated message, feel free to ignore.


Read the Docs

When I visit the http://neo4jrb.readthedocs.org/en/latest/ site, it show an empty page with empty content.

This is something that I've been struggling with with RTD for a while.  I hunk it stems from the fact that we have a really old git tag with the name "latest".  If you change "latest" to "stable" it should give you the docs.  Let's keep this issue though until we find a permanent solution

On Jun 24, 2015, at 07:10, Matthew Harmon<EMAIL_ADDRESS>wrote:
When I visit the http://neo4jrb.readthedocs.org/en/latest/ site, it show an empty page with empty content.
—
Reply to this email directly or view it on GitHub.

It's fixed now, but it's gone back to broken in the past.  I'll check it again in a week or so

Sounds good.
BTW, I think there should be a cookbook with more explicit examples on how to use this gem. The current documentation got me started with a project, but it took a lot of trial and error to get it working. Such as finding out that find_or_create method not throwing errors when it didn't do anything.
I might open a separate issue, but do you know if this 5.0 version will have factory girl support?

Good point, I've added #851.  Also note that I'm planning on releasing a series of short screencasts which will try to thoroughly overview each aspect of each functionality.
As far as factory_girl, that's something I would definitely like to see.  I don't think it would work right now, but @subvertallchris might have a better idea

Right now I am using: gem 'neo4j', github: 'dpisarewski/neo4j', branch: 'factory_girl', which is version 4.0.0.rc.1

Oh, nice, so it seems like that's just a month or two old.  @dpisarewski what's the status of that branch?  Are you still using it?  If so, it would be nice to get it into master

@mattdharmon @cheerfulstoic It looks like that link loads properly and this can be closed.

Ah, yes.  I think I created a latest tag, though that's stuck as a particular commit and thus will get older and older.  http://neo4jrb.readthedocs.org/ is the best URL in general and right now that forwards to http://neo4jrb.readthedocs.org/en/5.2.x/  We'll change the default version to whatever the newest version branch is each time we update the gem(s)


Add better documentation for #select

@ejholmes this should be a quick win.  In response to #182 I'm hoping to clarify the #select method.

@theSteveMitchell :+1: (I'll make some stylistic changes during my merge)


TBC Timewalking Dungeon Quest: Updated ID

The quest ID for the TBC dungeon quest, "A Burning Path Through Time", has changed to have a Dornogal turn-in now.
                -- TBC
                {
                    key = 'timewalkingDungeons',
                    minimumLevel = 70,
                    requiredEventIds = tbcTimewalkingIds,
                    entries = {
                        { quest = 83363 }, -- A Burning Path Through Time
                    },
                },

This was already updated and released in v<IP_ADDRESS>


Contractions usage

Description

Adds more guidance, as requested, around usage of contractions.
Removed explanation of contractions to get straight to the style guidance.

Additional information

Will close #123

Definition of Done

[ ] PR approved by Libby
[ ] Related GitHub issues closed
[ ] Update to Updates page

Hey @fearlesssteven can you draft the abbreviations edit in this branch (bugfix/123-contractions) as per #123? Happy to help out with this.

Oh never mind - sorry I see now what you did :) @fearlesssteven

Hi @libbyv another for your review (Contractions and Abbreviations & acronyms entries). I may be able to link you to a staged site to review this soon.

Hey @petronbot can you give me a hand with a cf push issue related?


HADOOP-17049. javax.activation-api and jakarta.activation-api define overlapping classes

JIRA: https://issues.apache.org/jira/browse/HADOOP-17049
Remove javax.activation-api from dependency.

jakarta.activation-api is the new name for javax.activation-api since 1.2.1, and we include jakarta.activation-api because we updated onto jackson-databind 2.10


Variable referenced before assignment error after updating to 0.12

liyzho01@e105699-lin:/work/iot/nrf51-dfu-bootloader-ble$ yt build
info: get versions for mbed-hal-nrf51dk
info: download<EMAIL_ADDRESS>from the public module registry
info: generate for target: nrf51dk-gcc 1.0.0 at /work/iot/nrf51-dfu-bootloader-ble/yotta_targets/nrf51dk-gcc
Fatal Exception, yotta=0.12.0
Traceback (most recent call last):
  File "/usr/local/bin/yt", line 4, in <module>
    yotta.main()
  File "/usr/local/lib/python2.7/dist-packages/yotta/main.py", line 205, in main
    status = args.command(args, following_args)
  File "/usr/local/lib/python2.7/dist-packages/yotta/build.py", line 46, in execCommand
    status = installAndBuild(args, following_args)
  File "/usr/local/lib/python2.7/dist-packages/yotta/build.py", line 120, in installAndBuild
    for error in generator.generateRecursive(c, all_components, builddir, application=app):
  File "/usr/local/lib/python2.7/dist-packages/yotta/lib/cmakegen.py", line 131, in generateRecursive
    c, all_components, os.path.join(modbuilddir, name), modbuilddir, processed_components, application=application
  File "/usr/local/lib/python2.7/dist-packages/yotta/lib/cmakegen.py", line 131, in generateRecursive
    c, all_components, os.path.join(modbuilddir, name), modbuilddir, processed_components, application=application
  File "/usr/local/lib/python2.7/dist-packages/yotta/lib/cmakegen.py", line 131, in generateRecursive
    c, all_components, os.path.join(modbuilddir, name), modbuilddir, processed_components, application=application
  File "/usr/local/lib/python2.7/dist-packages/yotta/lib/cmakegen.py", line 131, in generateRecursive
    c, all_components, os.path.join(modbuilddir, name), modbuilddir, processed_components, application=application
  File "/usr/local/lib/python2.7/dist-packages/yotta/lib/cmakegen.py", line 131, in generateRecursive
    c, all_components, os.path.join(modbuilddir, name), modbuilddir, processed_components, application=application
  File "/usr/local/lib/python2.7/dist-packages/yotta/lib/cmakegen.py", line 122, in generateRecursive
    self.generate(builddir, modbuilddir, component, new_dependencies, dependencies, recursive_deps, application, toplevel)
  File "/usr/local/lib/python2.7/dist-packages/yotta/lib/cmakegen.py", line 501, in generate
    "module_is_empty": module_is_empty,
UnboundLocalError: local variable 'module_is_empty' referenced before assignment

seems to have been introduced here: https://github.com/ARMmbed/yotta/commit/27568784a6b1928cbf138a0367454fe424c78843

:thumbsup:

(Note: before closing this, make sure that building with custom CMakeLists.txt is covered in tests, seems not to be!)


On page load "InitItem JSON.parse error in bindfeilds.js line 8"

You MUST use this template when reporting issues. Please make sure you follow the checklist and fill in all of the information sections below.

All versions of django-smart-selects prior to version 1.2.8 are vulnerable to an XSS attack as detailed in issue 171. As a result, all previous versions have been removed from PyPI to prevent users from installing insecure versions. All users are urged to upgrade as soon as possible.
Checklist
Put an x in the bracket when you have completed each task, like this: [x]

[x] This issue is not about installing previous versions of django-smart-selects older than 1.2.8. I understand that previous versions are insecure and will not receive any support whatsoever.
[x] I have verified that that issue exists against the master branch of django-smart-selects.
[x] I have searched for similar issues in both open and closed tickets and cannot find a duplicate.
[x] I have debugged the issue to the smart_selects app.
[x] I have reduced the issue to the simplest possible case.
[x] I have included all relevant sections of models.py, forms.py, and views.py with problems.
[x] I have used GitHub Flavored Markdown to style all of my posted code.

Steps to reproduce

Have installed smart-selects as per instructions
When load issue appeared have manually loaded all js files as per issue #214
When page loads get InitItem JSON.parse error in bindfeilds.js line 8 similar to issue #227

Models (have been reduced to just important fields)
class Vocation(Audit):
    CAT_CHOICES = (
        ('GENERAL', 'General'),
        ('ELEMENTARY', 'Elementary'),
        ('SECONDARY', 'Secondary'),
        ('PRESCHOOL', 'Preschool'),
        ('DISTRICT', 'District'),
    )

    name = models.CharField(max_length=255)
    category = models.CharField(max_length=255, choices=CAT_CHOICES)

    def __str__(self):
        return self.name

    class Meta:
        verbose_name_plural = "vocations"

class Area(Audit):
    name = models.CharField(max_length=255)
    vocations = models.ManyToManyField(Vocation)

    def __str__(self):
        return self.name

    class Meta:
        verbose_name_plural = "areas"

class Employee(Audit):

    user = models.OneToOneField(User, on_delete=models.CASCADE, related_name='employee')
    cell_phone = PhoneNumberField(blank=True, null=True)
    gender = models.CharField(max_length=10, choices=GENDER_CHOICES, blank=True, null=True)
    hired_by = models.CharField(max_length=10, choices=HIRED_CHOICES, blank=True, null=True)
    hire_type = models.CharField(max_length=20, choices=HIRE_TYPE_CHOICES, blank=True, null=True)
    buildings = models.ManyToManyField(Building)
    avatar = models.ImageField(upload_to=avatar_upload_path, blank=True, null=True)
    reviewed_date = models.DateTimeField(blank=True,null=True)

    #TODO get ChainedManytoManyField to work as expected
    position = models.ForeignKey(Vocation, blank=True, null=True)
    area = ChainedManyToManyField(
        Area,
        horizontal=True,
        verbose_name='areas',
        chained_field="position",
        chained_model_field="vocations")

    class Meta:
        verbose_name_plural = "employees"

    def __str__(self):
        return self.full_name

Forms
import datetime

from django import forms
from .models import Employee

class EmployeeForm(forms.ModelForm):

    class Meta:
        model = Employee
        fields =['gender', 'cell_phone','buildings', 'hired_by',
                         'hire_type', 'position','area']

View
class EmployeeFormView(UpdateView):
    model = Employee
    template_name = 'ironcsd/employee_form.html'
    form_class = EmployeeForm
    success_url = reverse_lazy('profile-home')

    def get_object(self, queryset=None):
        obj, created = Employee.objects.get_or_create(
                    user=self.request.user.id,
                )
        return obj

    def form_valid(self, form):
        object = form.save(commit=False)

        object.reviewed_date = datetime.datetime.now()
        object.activity_user = self.request.user.username
        object.activity_date = datetime.datetime.now()
        object.save()

        return super(EmployeeFormView,self).form_valid(form)

Actual behavior

As the bindfields.js does not load properly and errors out at fetching the data-value the area field select does not populate when a position/vocation is selected.
Sometimes at random the bindfields.js loads properly and will load the chained data to the dom. This is completely at random and have not been able to troubleshoot the reason for data getting set as expected.

Expected behavior

To work as plugin is expected to work.

No feedback, closing.


[Request] -- Host Option

Hey! I'm coming from the old ws repo since I heard all the goodies about this and have been testing it. Performance is insane.
One thing I've noticed though, is there is no host option available as from the old ws, for example:
var wss = new WebSocketServer({ host: '<IP_ADDRESS>', port: 9300 });
This is useful as it doesn't expose the port, only to 127.0.01 (locally). in uWebSockets (this module), it exposes it to the outside -- You can use iptables to fix this, but meh. I think if there was an added host option, similar to ws, it would be great!
This is useful when sticking the uws server behind a reverse proxy as well!

Sure

So basically this is just about taking the host option and passing it to listen: https://nodejs.org/api/http.html#http_server_listen_port_hostname_backlog_callback. This is up for grabs.

I guess this solves it: 0ab94acee26967260309e60663a5d890d7d7c7dd


Containerize the back end

Description:

Problem:
We need to containerize the back end to make it work on Apple devices.

Solution:
Use docker and add a compose.yaml file for all the back-end services.

Prerequisites:
[List any requirements or dependencies needed before starting.]

Tasks:

Containerize everything related to the card gen service

Additional context
Add any other context or screenshots about the feature request here.

@TimKoornstra @WinstonLam Is this really something we need to do?

It does make it easier for users to run it themselves

@TimKoornstra would running docker be easier compared to Python? Only upside I can see is that the chances of them messing up the commands or having a broken machine are smaller

Docker ensures an environment that should always work and can be run with a single command, instead of having to navigate through everything manually and installing things

Okay we'll keep it open for now, @WinstonLam has already made a start in the branch feature/card-gen-containerization


Mention required platform target versions in docs

iOS: 11
macOS: 10.13
Android: 19

Closed by 4d2e33c135e9494a3537e57e93ea3a41946ede44, 6dd32c4dbca8f2ba25d072344625cc100661c68b


IDs can be bytes as well as strings

more detailed fix here https://github.com/graphprotocol/docs/pull/106