Extra logging added to Maven builds makes the build log hard to read

In our project we have the Maven Integration Plugin - 3.6 and Jenkins - 2.222.3 installed, and I'm seeing org.apache.maven.cli.event.ExecutionEventLogger everywhere in Jenkins Console logs only in the publish stage as below.
[Pipeline] { (Publish) (hide)
[Pipeline] newArtifactoryServer
[Pipeline] newBuildInfo
[Pipeline] newMavenBuild
[Pipeline] artifactoryMavenBuild
02:51:23  Jenkins Artifactory Plugin version: 3.6.2
02:51:23  Artifactory integration is enabled
02:51:25  [feature-test] $ java -classpath /usr/share/maven/boot/* -Dmaven.home=/usr/share/maven -Dmaven.conf=/usr/share/maven/conf
<EMAIL_ADDRESS>-DbuildInfoConfig.artifactoryResolutionEnabled=true<EMAIL_ADDRESS>-Dclassworlds.conf=/home/jenkins/workspace/_feature-test@tmp/artifactory/classworlds5581946743250217473conf
-Dmaven.multiModuleProjectDirectory=/home/jenkins/workspace/_feature-test -Xmx1536m org.codehaus.plexus.classworlds.launcher.Launcher
-f pom.xml install package -DskipTests
02:51:26  [main] INFO org.apache.maven.cli.event.ExecutionEventLogger - Scanning for projects...
02:51:26  [main] INFO org.jfrog.build.extractor.maven.BuildInfoRecorder - Initializing Artifactory Build-Info Recording
02:51:26  [main] INFO org.apache.maven.cli.event.ExecutionEventLogger -
02:51:26  [main] INFO org.apache.maven.cli.event.ExecutionEventLogger - ---------------< samples-java-library >----------------
02:51:26  [main] INFO org.apache.maven.cli.event.ExecutionEventLogger - Building devops-java-basic 1.1.0-feature-test.1
02:51:26  [main] INFO org.apache.maven.cli.event.ExecutionEventLogger - --------------------------------[ jar ]---------------------------------
02:51:26  Downloading from artifactory-release: https://com/artifactory/maven-dev/org/apache/maven/plugins/maven-plugins/23/maven-plugins-23.pom
02:51:26  Downloaded from artifactory-release: https://com/artifactory/maven-dev/org/apache/maven/plugins/maven-plugins/23/maven-plugins-23.pom (0 B at 0 B/s)
02:51:27  [main] INFO org.apache.maven.cli.event.ExecutionEventLogger -
02:51:27  [main] INFO org.apache.maven.cli.event.ExecutionEventLogger - --- maven-resources-plugin:2.6:resources (default-resources) @ samples-java-library ---
02:51:27  Downloaded from artifactory-release: https://com/artifactory/maven-dev/org/apache/apache/13/apache-13.pom (0 B at 0 B/s)
02:51:27  Downloading from artifactory-release: https://com/artifactory/maven-dev/org/apache/maven/plugins/maven-compiler-plugin/3.1/maven-compiler-plugin-3.1.jar
02:51:27  Downloaded from artifactory-release: https://com/artifactory/maven-dev/org/apache/maven/plugins/maven-compiler-plugin/3.1/maven-compiler-plugin-3.1.jar (0 B at 0 B/s)
02:51:27  Downloading from artifactory-release: https://com/artifactory/maven-dev/org/apache/maven/plugins/maven-surefire-plugin/2.12.4/maven-surefire-plugin-2.12.4.pom
02:51:27  Downloading from artifactory-release: https://com/artifactory/maven-dev/org/apache/maven/plugins/maven-surefire-plugin/2.12.4/maven-surefire-plugin-2.12.4.pom
Makefile for package and publish are as below and the publish stage og groovy is shared below,
package:
mvn -B $(MVN_VERBOSE_ARGS) package -DskipTests
publish:
@echo "publish"
@Override
def publish(config) {
ArtifactoryUtils artifactoryUtils = new ArtifactoryUtils()
def server = artifactoryUtils.getArtifactoryServer(config.artifactoryBaseUrl, config.artifactoryUser, config.artifactoryPassword)
def buildInfo = artifactoryUtils.getBuildInfo()
artifactoryUtils.publishMavenLib(config.project, server, buildInfo)
artifactoryUtils.publishBuildInfo(server, buildInfo)
return buildInfo
}
Do we have any ideas on suppressing these artifactory logs in publish stage?

Unrelated to coursier


Close Combat True Damage

when will close combat damage be added?

He needs to add a default setting of distance in configuration tab, "Impact" from slayer also would get benefit from setting enemy distance.

I didn't find in Path of Building

https://pastebin.com/k1cfL7hM

I said that Openarl needs to add a setting of distance to make close combat work.
The code already exists and is showed on settings tab when using Point Blank with a Projectile skill, but needs to be implemented on melee skills too.
In your language: A opção de distância já existe mas apenas para projéteis usando point blank por exemplo, para calcular o dano baseado na distancia.
Mas ainda não foi implementado a opção de definir a distancia em habilidades corpo a corpo/melee, só podemos esperar pois ele provavelmente está muito ocupado com a 4.0 já que ele agora trabalha pra GGG.
O que eu citei ali em cima foi que a ascendência do Slayer tem um bônus baseado em distância também que não consta no Path of Building devido a falta dessa configuração de distância, então seria muito útil se ele adicionasse.
O texto em vermelho abaixo | Text in red

https://pathofexile.gamepedia.com/Impact

He needs to add a default setting of distance in configuration tab, "Impact" from slayer also would get benefit from setting enemy distance.

I said that Openarl needs to add a setting of distance to make close combat work.
The code already exists and is showed on settings tab when using Point Blank with a Projectile skill, but needs to be implemented on melee skills too.

https://pathofexile.gamepedia.com/Impact

I created PR https://github.com/Openarl/PathOfBuilding/pull/1670 to resolve this. :)

After digging in further on the Close Combat gem (stat "support_slashing_damage_+%_final_from_distance" specifically), it looks like another formula (similar to "DistanceRamp", except for melee). On https://pathofexile.gamepedia.com/Close_Combat_Support, I found that it is full bonus at 15 units, scaling down in a linear fashion to 40 units. I'll cancel my original PR, correct this, and then resubmit one for Impact and Close Combat support.

@igorcastroo,
Created PR https://github.com/Openarl/PathOfBuilding/pull/1674 to implement both properly. Hoping Openarl will see, review, and approve/deny soon. :)

@igorcastroo Created PR #1699 to cover this ( superseding #1674 ) and pave the way for much easier corrections/additions. Openarl is either super busy or uninterested in outside help. If that pull request is merged in, the entire Slayer ascendency will be functional and I can offer a lot more help. If not, guess it's stuck incomplete and time for me to disappear into the void again. </3


Feature: Allow SqliteDataReader.GetOrdinal to be case insensitive

I know that the parameter is case sensitive, however is there a way to allow for insensitive? The constructor is marked internal so I cannot extend the object and override the method. I am forced to loop through the column names twice right now. Once to do the case insensitive search and then then I use the reader["columnname"] this method will loop again. Or is there another way to accomplish this? The code I am using works fine with the sqlclient and the mono.data.sqlite implementation of the sqlitedatareader, but I would prefer to use this one if possible.
/// <remarks>The <paramref name="name" /> parameter is case sensitive.</remarks>
        public override int GetOrdinal(string name)
        {
            for (var i = 0; i < FieldCount; i++)
            {
                if (GetName(i) == name)
                {
                    return i;
                }
            }

            // NB: Message is provided by framework
            throw new ArgumentOutOfRangeException(nameof(name), name, message: null);
        }

We aren't planning to do this as we're currently just exposing what SQLite does natively.

Confirmed; SQLite is case-sensitive. Here is a query to demonstrate.
SELECT
    1 AS a,
    2 AS A;


Feedback: Keyboard covers email input

The keyboard covers the email input which makes it harder to type
https://user-images.githubusercontent.com/44020029/189093743-f022e17a-85dd-4070-9d79-85eb1880e45d.mp4

interestingly, I can reproduce this on my friends Android phone, but not on my iPhone 😄 I'll try to fix it anyway


System.Reflection.TargetInvocationException: 'Exception has been thrown by the target of an invocation.'

Describe the bug
Since updating a project to the latest version of WinUI3 (Reunion 0.5) we receive the following error whenever trying to debug:
System.Reflection.TargetInvocationException
  HResult=0x80131604
  Message=Exception has been thrown by the target of an invocation.
  Source=System.Private.CoreLib
  StackTrace:
   at System.RuntimeTypeHandle.CreateInstance(RuntimeType type, Boolean publicOnly, Boolean wrapExceptions, Boolean& canBeCached, RuntimeMethodHandleInternal& ctor, Boolean& hasNoDefaultCtor)
   at System.RuntimeType.CreateInstanceDefaultCtorSlow(Boolean publicOnly, Boolean wrapExceptions, Boolean fillCache) in /_/src/coreclr/src/System.Private.CoreLib/src/System/RuntimeType.CoreCLR.cs:line 4005
   at System.RuntimeType.CreateInstanceDefaultCtor(Boolean publicOnly, Boolean skipCheckThis, Boolean fillCache, Boolean wrapExceptions) in /_/src/coreclr/src/System.Private.CoreLib/src/System/RuntimeType.CoreCLR.cs:line 4062
   at System.Activator.CreateInstance[T]() in /_/src/libraries/System.Private.CoreLib/src/System/Activator.RuntimeType.cs:line 151
   at WinRT.WeakLazy`1.get_Value()
   at Microsoft.UI.Xaml.Application._IApplicationStatics.get_Instance()
   at Microsoft.UI.Xaml.Application.Start(ApplicationInitializationCallback callback)
   at Project.Program.Main(String[] args) in C:\Users\USER\Source\Repos\Project\Project.UI\Project.UI\obj\x64\Debug\net5.0-windows10.0.19041.0\App.g.i.cs:line 26

  This exception was originally thrown at this call stack:
    System.Runtime.InteropServices.Marshal.ThrowExceptionForHR(int) in Marshal.cs
    WinRT.BaseActivationFactory.BaseActivationFactory(string, string)
    Microsoft.UI.Xaml.Application._IApplicationStatics._IApplicationStatics()

Inner Exception 1:
COMException: Class not registered (0x80040154 (REGDB_E_CLASSNOTREG))

Steps to reproduce the bug
We used a project created in WinUI3 v3 and followed the official instructions to upgrade to Reunion 0.5. The original project works as expected until Reunion is installed/WinUI3 removed - then we are unable to run the project with the CLASSNOTREG error.
Expected behavior
The project will run as before.
Screenshots

Version Info
NuGet package version:
Microsoft.ProjectReunion-v0.5.0-prerelease
Microsoft.ProjectReunion.Foundation-v0.5.0-prerelease
Microsoft.ProjectReunion.WinUI-v0.5.0-prerelease
Windows app type:
Win32

Windows 10 version
Saw the problem?

Insider Build (xxxxx)

October 2020 Update (19042)
Yes

May 2020 Update (19041)

November 2019 Update (18363)

May 2019 Update (18362)

October 2018 Update (17763)

April 2018 Update (17134)

Fall Creators Update (16299)

Creators Update (15063)

Device form factor
Saw the problem?

Desktop
Yes

Xbox

Surface Hub

IoT

@fabiant3 and @RealTommyKlein Is the reflection provider your team's?

This is not from the reflection provider, as it doesn't exist for WinUI 3/Project Reunion.  As a sanity check, the app being debugged doesn't have "Package" in the name, which is the default for packaging projects (e.g. "ETIHub.TDHub.Demo.UI (Package)" instead of "ETIHub.TDHub.Demo.UI").  Can you confirm the wapproj/packaging project is being run instead of the unpackaged application?

@RealTommyKlein , lots of folks are making the mistake of running the wrong project.  I'm also baffled by the double compile cycle?  We've been down this road before, and always wind up with a single project.  Better sooner than later.

@RealTommyKlein - Unfortunately this is not the case (screenshot below):

For now we've gotten around the issue by creating a new project of type 'Blank App, Packaged (WinUI in Desktop)' and moved the contents of the old project in to it. This 'fresh' project runs without issue.

@s3nt1n3l-hub - good to hear you got it working.  Did you notice any differences in the working vs. non-working project that weren't covered by our upgrade instructions?

For now we've gotten around the issue by creating a new project of type 'Blank App, Packaged (WinUI in Desktop)' and moved the contents of the old project in to it. This 'fresh' project runs without issue.

I tried that, but I get the same error as described here, when creating a new project:

I just copy the global.json file and edit the csproj file (see here https://github.com/microsoft/ProjectReunion/issues/740) but the same error persists.


updated build files.

Changes to the master have been made since the last time the combined and minimized jsfeat.js files were last updated. Specifically hough_transform support has been added.
Recommend also tagging a new release for people to use.

There seems to be no conflicts. I'm using a copy of jsfeat in another project that I currently refactor.
I would like to remove that copy and instead require('jsfeat') directly.
I wrote a message to @inspirit  on twitter here: https://twitter.com/serapath/status/863139445184225280
If @inspirit doesn't maintain this anymore, maybe a fork can be created to include this and maybe the other pull request and maybe some more changes that @mccoder did here: https://github.com/mccoder/clmtrackr/commit/82686b07aa6f46906b88482aa21280dfc1164345
(He described those changes here: https://github.com/auduno/clmtrackr/issues/104#issuecomment-301176602)
It could be published under https://www.npmjs.com/package/js-feat

sorry seems like i did not receive notification for it for some reason...

awesome :-)
What about his one? https://github.com/inspirit/jsfeat/pull/18
There was some discussion and it looks like the @ghost added some commits but then nothing happened.
Is this desireable to be merged or out of scope or why did progress stop? :-)
I'd just would like to fork the most up2date version in order to make a pull request myself :-)
Would a pull request containing https://github.com/mccoder/clmtrackr/commit/82686b07aa6f46906b88482aa21280dfc1164345 be welcome?
It's about 2 files:

https://github.com/mccoder/clmtrackr/blob/dev/js/jsfeat_detect.js
https://github.com/mccoder/clmtrackr/blob/dev/js/jsfeat_detect_worker.js

@inspirit oh, and do you mind if i publish the module to npm?
This one: https://www.npmjs.com/package/jsfeat seems to be published by a different guy and a bit outdated too.

the one from #18 can not be merged unfortunately when i saw that pull request the user account was deleted @ghost is just empty name for deleted user. so if you want to have that added you can reopen request with existing user

I'm working on a quite serious refactoring of the project. I hope it will get merged.
If it's too much I understand and would just maintain a fork.
It's not done yet, because I have to update the demo's but you can preview already here:
https://github.com/serapath-contribution/jsfeat
The github-page is now served from the demo/ folder in the master branch and the "gh-pages" branch is removed. You can go to settings in github to change that.
I will try to make all the demos work now and then maybe a pull request makes sense? ...guess that's up to you :-)

so what is the reason for such changes?


⚠ ClamAV has degraded performance

In 5255892, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 74 ms

Resolved: ClamAV performance has improved in d6d3cc0 after 5 minutes.


Remove UTA Light Rail Routes from SGID

Summary

Internal Dataset Name: Transporation.LightRail_UTA
External Dataset Name: Utah UTA Light Rail Routes
ArcGIS Online Url: https://utah.maps.arcgis.com/home/item.html?id=fd41770e917344e4adef8be71a0a6741

UTA now has its own ArcGIS Enterprise Portal and is hosting this layer and sharing it with UGRC's SGID (SGID on ArcGIS) via Distributed Collaboration in UGRC's AGOL.
Migration Guide
The goal is to direct our users to UTA's web service, via the SGID Index and the SGID on ArcGIS.
Soft Delete
The purpose of the soft delete is to ensure that all of our users and applications have gracefully migrated off of the dataset. Soft deletes will remain in effect for 14 days. During this time, we will have the ability to restore the dataset to its original SGID offering(s). After these 14 days, the item is then ready for a hard delete.
Note: If this dataset is being replaced, then wait until the new data is publicly available before completing these steps:

[x] Append "(Mature Support)" to the end of the item title in the SGID.META.AGOLItems table (@gregbunce, completed: 2024/07/30)
[x] Remove all tags other than "Deprecated" in the SDE metadata (@gregbunce, completed: 2024/07/30)
[x] Add note to SDE description noting when layer will be deleted and any replacement layer (@gregbunce, completed: 2024/07/30)
[x] Update the SGID Index row to point to UTA's hosted data (@gregbunce, completed: 2024/07/30)
[x] Change Authoritative field to d in SGID.META.AGOLItems to automatically set the Deprecated AGOL flag. Allow the d to persist through one run of Auditor - currently, Auditor runs daily at 5:00am (@gregbunce, completed: 2024/07/30)

After at least one successful run of Auditor:

[x] Remove the row from the SGID.META.AGOLItems table. This will trigger the removal of this item in Open SGID (@gregbunce, completed: 2024/08/01)
[x] Unshare the item from its SGID group, which will remove it from the SGID on ArcGIS (@gregbunce, completed: 2024/07/30)

[ ] Check if this layer in the base maps. If so, work with Zach to develop a path forward before you proceed with a Hard Delete (@ZachBeck, completed: 2024/00/00).

 It is recommended to complete the soft delete process before moving on to these steps. If you decide to skip the soft delete, note that you will need to incorporate some of those steps here.

[ ] Manually remove data from the Internal SGID (@ZachBeck, completed: 2024/00/00)

[ ] Deprecated database layers should be backed up on Google Drive > AGRC Projects > SGID > deprecated layers > internal_db_sgid_layers (@ZachBeck, completed: 2024/00/00).

[ ] Remove ArcGIS Online item: (@ZachBeck, completed: 2024/00/00)

Manually delete the AGOL item
Deprecated AGOL items can be backed up on Google Drive > AGRC Projects > SGID > deprecated layers > agol_sgid_layers.

[ ] Remove Farm from AGOL connection (@stdavis, completed: 2024/00/00)
[ ] Update the SGID Index fields: "arcGisOnline", "openSgid", "openSgidTableName" (@gregbunce, completed: 2024/00/00)
[ ] Remove/delete the relevant gis.utah.gov data page (including downloadMetadata.ts) (@ZachBeck, completed: 2024/00/00)
[ ] Update SGID.META.AGOLItems table, remove row (@ZachBeck, completed: 2024/00/00)
[ ] Remove row from SGID.META.ChangeDetection (@stdavis, completed: 2024/00/00)
[ ] Remove data from forklift hashing and receiving (@stdavis, completed: 2024/00/00)
[ ] Remove row from data/hashed/changedetection.gdb/TableHashes (@stdavis, completed: 2024/00/00)

Shelve/Static
Choose one based on situation.
~- [ ] Upload to UtahAGRC/AGRC_Shelved folder in AGOL (New shelved item not already in AGOL) (name, completed: 2024/00/00)~
~- [ ] Move existing AGOL item to AGRC_Shelved AGOL folder (shelving an item already in AGOL) (name, completed: 2024/00/00)~
~- [ ] Upload to appropriate UtahAGRC/{SGID Category} folder in AGOL (for static datasets) (name, completed: 2024/00/00)~
Add record to table.
~- [ ] Add record to AGOLItems_shelved table in ArcGIS Online with shelved or static in the CATEGORY field (name, completed: 2024/00/00)~
:robot: Automation validation

Assign yourself or someone to check the item by replacing name with their github @name.
Check [x] the box and add the date of verification 2020/01/01 when the task is verified.
~Strike~ out all items that do not apply.

[ ] Remove data from Open SGID (@steveoh, on 2024/00/00)
[ ] sgid-index (@steveoh on 2024/00/00)
[x] Auditor sets appropriate shelved/static/Deprecated information (@jacobdadams on 2024/07/31)

Are there service dependencies

[ ] Are clients querying this data with the WebAPI?
[ ] Notify Don Jackson (don.jackson1 at motorolasolutions.com) if the data is in the Next-Generation 911 Motorola Aware Map? (@gregbunce on 2024/00/00)
[ ] Does any other application depend on this data?

Notification

[ ] Twitter (@steveoh)

Group Task Assignments

Check [x] the box when you have assigned all the tasks relevant to your group.

[x] Data Team (@gregbunce)
[ ] Dev Team (@steveoh)
[ ] Cadastre Team (@rkelson)

conductor results for tasks - 333

check
status

@gregbunce has completed 1 out of 10 tasks
:no_entry:

@ZachBeck has completed 0 out of 6 tasks
:no_entry:

@stdavis has completed 0 out of 4 tasks
:no_entry:

@steveoh has completed 0 out of 4 tasks
:no_entry:

@jacobdadams has completed 0 out of 1 tasks
:no_entry:

@rkelson has completed 0 out of 1 tasks
:no_entry:

conductor results for Transporation.LightRail_UTA

check
status

internal sgid
:+1:

meta table
:+1:

stewardship

- porter issue link
:no_entry:

conductor results for tasks - 333

check
status

@gregbunce has completed 8 out of 10 tasks
:no_entry:

@ZachBeck has completed 0 out of 6 tasks
:no_entry:

@stdavis has completed 0 out of 4 tasks
:no_entry:

@steveoh has completed 0 out of 4 tasks
:no_entry:

jacobdadams has completed 1 out of 1 tasks
:+1:

@rkelson has completed 0 out of 1 tasks
:no_entry:

conductor results for Transporation.LightRail_UTA

check
status

internal sgid
:+1:

meta table
:+1:

stewardship

- porter issue link
:no_entry:

conductor results for tasks - 333

check
status

@gregbunce has completed 8 out of 10 tasks
:no_entry:

@ZachBeck has completed 0 out of 6 tasks
:no_entry:

@stdavis has completed 0 out of 4 tasks
:no_entry:

@steveoh has completed 0 out of 4 tasks
:no_entry:

jacobdadams has completed 1 out of 1 tasks
:+1:

@rkelson has completed 0 out of 1 tasks
:no_entry:

conductor results for Transporation.LightRail_UTA

check
status

internal sgid
:+1:

meta table
:+1:

stewardship

- porter issue link
:no_entry:

conductor results for tasks - 333

check
status

gregbunce has completed 10 out of 10 tasks
:+1:

@ZachBeck has completed 5 out of 6 tasks
:no_entry:

@stdavis has completed 0 out of 5 tasks
:no_entry:

@steveoh has completed 0 out of 4 tasks
:no_entry:

jacobdadams has completed 1 out of 1 tasks
:+1:

@rkelson has completed 0 out of 1 tasks
:no_entry:

conductor results for Transporation.LightRail_UTA

check
status

internal sgid
:+1:

meta table
:+1:

stewardship

- porter issue link
:no_entry:

note about this layer and the sgid index: I used the existing row (record) for this layer in the sheet and updated the values with the newest information (name, urls, etc).

conductor results for tasks - 333

check
status

gregbunce has completed 10 out of 10 tasks
:+1:

@ZachBeck has completed 5 out of 6 tasks
:no_entry:

@stdavis has completed 1 out of 5 tasks
:no_entry:

@steveoh has completed 2 out of 4 tasks
:no_entry:

jacobdadams has completed 1 out of 1 tasks
:+1:

@rkelson has completed 0 out of 1 tasks
:no_entry:

conductor results for Transporation.LightRail_UTA

check
status

internal sgid
:+1:

meta table
:+1:

stewardship

- porter issue link
:+1:

conductor results for tasks - 333

check
status

gregbunce has completed 10 out of 10 tasks
:+1:

ZachBeck has completed 6 out of 6 tasks
:+1:

stdavis has completed 4 out of 4 tasks
:+1:

@steveoh has completed 2 out of 4 tasks
:no_entry:

jacobdadams has completed 1 out of 1 tasks
:+1:

@rkelson has completed 0 out of 1 tasks
:no_entry:

conductor results for Transporation.LightRail_UTA

check
status

internal sgid
:+1:

meta table
:+1:

stewardship

- porter issue link
:+1:

Do we want the SGID index item to have the word "Routes" in the title @gregbunce?

conductor results for tasks - 333

check
status

gregbunce has completed 10 out of 10 tasks
:+1:

ZachBeck has completed 6 out of 6 tasks
:+1:

stdavis has completed 4 out of 4 tasks
:+1:

steveoh has completed 4 out of 4 tasks
:+1:

jacobdadams has completed 1 out of 1 tasks
:+1:

@rkelson has completed 0 out of 1 tasks
:no_entry:

conductor results for Transporation.LightRail_UTA

check
status

internal sgid
:+1:

meta table
:+1:

stewardship

- porter issue link
:+1:


Console not working with Primate UI

ISSUE TYPE

Bug Report

COMPONENT NAME

UI

CLOUDSTACK VERSION

<IP_ADDRESS>

CONFIGURATION

OS / ENVIRONMENT

CentOS 7
SUMMARY

Upgrading from Cloudstack version <IP_ADDRESS> to <IP_ADDRESS>, if you use the primate interface and try to open the VNC console on any instance (no matter the privileges of the user account) it does not work and you obtain only "failed" message.
STEPS TO REPRODUCE

Upgrade from CS<IP_ADDRESS> to <IP_ADDRESS> and use the Primate interface. The VNC console won't be accessible.

EXPECTED RESULTS

VNC console normally working as in version <IP_ADDRESS>

ACTUAL RESULTS

Nothing happens and you receive "failed" message

@cdfgallo since 4.18.0 added API driven console, can you please check if you are getting any error in the management server logs?
cc @nvazquez

@cdfgallo I can confirm that in 4.18.0 there is a problem but this is solved in 4.18.1. If you upgrade and recreate the console proxy, it should work.

@cdfgallo since 4.18.0 added API driven console, can you please check if you are getting any error in the management server logs?
cc @nvazquez

unfortunately I can't find any log on the management server nor on the CPVM

Hi @cdfgallo can you increase the log level on the log4j.xml configuration file on the CPVM from INFO to DEBUG? After that, can you retry accessing consoles and check if something is logged on the cloud.log file on the CPVM?

@cdfgallo cc @nvazquez you may also check if createConsoleEndpoint API is executing fine

@cdfgallo I can confirm that in 4.18.0 there is a problem but this is solved in 4.18.1. If you upgrade and recreate the console proxy, it should work.

@cdfgallo @shwstppr @nvazquez I have reproduced the error in 4.18.0 and seen it was gone after upgrading to 4.18.1. I didn't check what PR fixed it but closing this as it is fixed.

Hello @DaanHoogland, I just upgrade my CS installation to version 4.18.1 but I can confirm you that I'm facing the same issue on this version too.
I tried re-create the CPVM (which is working with the new UI) but it still do not work
From the management node I found only this log
2023-10-29 09:34:22,378 DEBUG [c.c.u.AccountManagerImpl] (qtp1444635922-3124:null) (logid:) Access to VM instance {"id":298,"instanceName":"i-6-298-VM","type":"User","uuid":"bd0b24cf-0ede-45d0-a857-63e4e292f1dc"} granted to Account [{"accountName":"xxxx","id":141,"uuid":"f5b5cab6-025e-489c-9a4e-efde296bdcd3"}] by DomainChecker
2023-10-29 09:34:22,385 WARN  [c.c.s.ConsoleProxyServlet] (qtp1444635922-3124:null) (logid:) Null sid in URL.

ok @cdfgallo , so there is probably something extra going on. I'll reopen the issue.

@nvazquez @JoaoJandre , could you have a look as you are probably the most recent developers on the console proxy?

Hey @cdfgallo,
In order to troubleshoot your problem we will need a little bit of context:

Have you validated that you have access to the CPVM? Can you ping it from your machine?
Could you follow @nvazquez instructions here https://github.com/apache/cloudstack/issues/8113#issuecomment-1770726550 and post the MGMT and CPVM logs from the process that you are trying to do?

Hey @cdfgallo, In order to troubleshoot your problem we will need a little bit of context:

Have you validated that you have access to the CPVM? Can you ping it from your machine?
Could you follow @nvazquez instructions here Console not working with Primate UI #8113 (comment) and post the MGMT and CPVM logs from the process that you are trying to do?

Hello @JoaoJandre ,
yes, I have access to the CPVM, please note that this issue is present only from the primate UI.
I set the log level on DEBUG in the log4j-cloud.xml file, but unfortunatelly there are still no logs when I try to open the VNC console from the primate interface. From the MGMT server the only log I see is this one
"
2023-10-29 09:34:22,378 DEBUG [c.c.u.AccountManagerImpl] (qtp1444635922-3124:null) (logid:) Access to VM instance {"id":298,"instanceName":"i-6-298-VM","type":"User","uuid":"bd0b24cf-0ede-45d0-a857-63e4e292f1dc"} granted to Account [{"accountName":"xxxx","id":141,"uuid":"f5b5cab6-025e-489c-9a4e-efde296bdcd3"}] by DomainChecker
2023-10-29 09:34:22,385 WARN [c.c.s.ConsoleProxyServlet] (qtp1444635922-3124:null) (logid:) Null sid in URL.
"

Hey @cdfgallo,
When you click on the button to open the console. The UI should call the API createConsoleEndpoint. And this API should log something on the management server. So that we are on the same page, can you please do the following test:

On your browser, open your browser console (F12 on Firefox and Chrome) and go to the network tab.
Click on the button to open the VM console. You should see a new entry on the network log that refers to the API call that the UI is making. Can you confirm that the API is being called? If so, could you share the contents of the API request?
If you confirm that the API is being called, you should get at least a few logs of the process. You could try using something like grep -i 'createConsoleEndpoint' <mgmt-log-file> to get the start of the process's logs. From here, you can grep using the logid of the process to get the rest of the logs.

If you make it to the end of step 3. Could you please share the logs that you captured?

Hi @JoaoJandre ,
I've done the test you required and I can confirm you that, with the primate UI, the API createConsoleEndpoint is not being called.
Also from the management server I don't see any logs related to this.
Are there any other tests I can do for this issue?

Hello @cdfgallo,
I've tried to reproduce your problem on a lab running on version <IP_ADDRESS>, but I was unable to.
On my setup, the UI calls the API correctly and I can access the console.

Can you repeat the test using an anonymous tab?

Hello @JoaoJandre ,
unfortunately I can confirm you the error is the same using an anonymous tab.
Here you can find a screen when I open the console on primate UI.

Hi @JoaoJandre ,
I open the console accessing the VM and then clicking on "view console" button.
Yes, indeed I see that the URL is quite different, but it used to work with version 4.17.2

Hi @JoaoJandre ,
I open the console accessing the VM and then clicking on "view console" button. Yes, indeed I see that the URL is quite different, but it used to work with version 4.17.2

Hello @cdfgallo,
I can see from your screenshot that you're not using the GUI that is embedded with the community version. Otherwise, your UI should look something like this:

PR #6577 changed the way consoles are managed in ACS; therefore, unfortunately I'll not be able to help you with a custom deployment. You might try contacting your vendor to get further help, if it is the case.

Hi @cdfgallo, as @JoaoJandre points out the UI seems to be missing the changes regarding the CloudStack UI about the latest console access enhancements.
As a workaround, you can use the createConsoleEndpoint API to generate an access URL and paste it in your browser.
Example:
(localcloud) 🐱 > create consoleendpoint virtualmachineid=27b3a356-cc0d-451f-8c20-0aca0b60487a
{
  "consoleendpoint": {
    "success": true,
    "url": "http://xx.xx.xx.xx/resource/noVNC/vnc.html?autoconnect=true&port=8080&token=<TOKEN>",
    "websocket": {
      "host": "<HOST_IP>",
      "path": "websockify",
      "port": "8080",
      "token": "<TOKEN>"
    }
  }
}

Then copy the url field from the response in your browser to access the console. Can you try this approach?

Hi @weizhouapache, I'm sorry for the late reply.
We are not using cloudstack-ui package.
We found a workaround for this so we can close this issue as I see it's related to a old customization.
thank you all for the support.


DOWNLOAD FAILED

You see many other issues related to this one, yet I see the problem has NOT been resolved. Many people say that they're O3DS users, while I am getting the same issue on my N3DSXL. Whether I use my home's WiFi, My mom's Wifi, Turn my phone into a hotspot, it always says download failed...
Like why would it find a new version, if it can't download it???
I should also mention that I am on 11.4
Current version: 7.0.5-e029458b
Latest version (Github) 7.1

7.1 only works on boot9strap consoles. If you have a9lh, follow this to update to boot9strap. 7.0.5 is the last update for non-boot9strap consoles.


Detect multiple monitors using Xinerama extension for fullscreen

This is my second attempt to fix #1408. I tested in Ubuntu 14.10 with the
Unity desktop, LXDE, and OpenBox and it worked as expected in all three.
This change includes the Xinerama extension and queries it when going
fullscreen. Xinerama has information about the actual monitors
connected in a multi-monitor configuration. X normally treats both
monitors as a single large screen. Xinerama allows us to get the
actual dimensions of attached screens. Nvidia TwinView and other
drivers use the Xinerama protocol so this should be compatible with
most multi-screen setups.
This forces fullscreen to always go to screen 0. This fixes #1408.
This also fixes the issue in some window managers (e.g. LXDE and
OpenBox) where Godot would stretch across both monitors.

Sorry, this pull was from the wrong branch. I've replaced it with this one: https://github.com/okamstudio/godot/pull/1502


Receive domains as input for internetdb-api

Please describe your feature request:

Internal logic to resolve domain, use as input for internetdb, return all ports for all ips for that domain, ( https://github.com/projectdiscovery/naabu/issues/293 )
echo google.com | uncover -json

{
  "timestamp":1648469130,
  "source":"shodan-internetdb",
  "ip":[
    "<IP_ADDRESS>",
    "<IP_ADDRESS>"
  ],
  "ports":[
    80,
    443
  ],
  "cpes":[
    "cpe:/a:igor_sysoev:nginx",
    "cpe:/a:openbsd:openssh:7.4"
  ],
  "host":[
    "google.com"
  ],
  "tags":[
    ""
  ],
  "vulns":[
    ""
  ]
}

Describe the use case of this feature:

subfinder -d TARGET | uncover -json | jq '.host[]+":"+(.ports[]|tostring)' |httpx

https://github.com/projectdiscovery/naabu/issues/293
this idea will be implemented in naabu, being redundant here


"react-native-google-signin": "^1.0.0-rc7", react-native-version:0.59.9 every android or ios version working but only issue android with android version 29 please help me what i can do with that error

Steps to Reproduce

You must provide a minimal and easy-to-run reproduction of your issue - how to create a minimal reproduction?. We're a small team of maintainers and do not have time to try reproduce bugs ourselves.

Please try to reproduce the bugs on the provided example app. Either provide a link to the repo that reproduces the bug or provide your version of the index.js file that reproduces the issue and that we can easily use.

Explain the steps we need to take to reproduce the issue.

Include a gif if you think it may help.

If you're asking for help, please make it easy for us to help you!

Expected Behavior

Actual Behavior

Environment
Please provide

react-native version
@react-native-community/google-signin version
your Podfile, output of running pod install and cocoapods version (if applicable)
version of Android, gradle, android plugin for gradle, play-services-auth and version of the Google Play Services app on your phone (if applicable)
version of iOS (if applicable)

Hi there! Sorry you're having an issue. As the issue template explains, we require that you provide a runnable example (please try reproducing the bug with our example app) that reproduces your issue in a github repository. Please try to minimize the superfluous code and focus only on reproducing the bug.
please create a new issue with this and we'll be happy to review it!


Improve error messages when missing identity cookies

Description

The goal of this ticket was to improve the error messages when a identity cookie is not set after a Konductor response. We identified two potential causes for this

The provided organization ID is incorrect

new error message: "An identity for organization adobeOrg@Adobe was not found. Valid organizations on this page are: adobeOrg2@Adobe, adobeOrg3@Adobe"

The current apex domain doesn't allow third-party cookies from the edge domain.

new error message: "An identity was not set properly because edge domain example.com does not match apex domain adobe.com, and adobe.com may not allow third-party cookies."

("example.com" is used as an example of an edge domain and "adobe.com" is used as an example of an apex domain).
In addition to these two new error messages, the catch-all error message has been updated. It now reads as "An identity was not set properly. Please verify that cookies returned from demdex.net can be set on this page."
Related Issue

PDCL-7279–Better error message when Web SDK can't find identity cookie

Currently, when an interact request goes out where the Web SDK expects to get an identity cookie, if the cookie is not there, this message is generated:
"An identity was not set properly. Please verify that the org ID *** configured in Alloy matches the org ID specified in the edge configuration."
This message can lead users down the wrong path. We should actually look for cookies starting with kndctr_ and see if there is an org ID mismatch.
This problem could also be caused by collecting using a CNAME that is not first party to the current location. Can we check to see if the edgeDomain ends with the apex domain of the current location? We could log a warning in such a case when we don't get the identity cookie back.
When neither of these problems are found how about a message like this: "An identity was not set properly. Please verify the cookies returned from ${edgeDomain} can be set on this page."

Motivation and Context

Previous customers have found that the existing error message was not actionable. These new error message provide more useful next steps for the customer.
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Improvement (non-breaking change which does not add functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have signed the Adobe Open Source CLA or I'm an Adobe employee.
[x] I have made any necessary test changes and all tests pass.
[x] I have run the Sandbox successfully.

Failing functional tests will be resolved when pull request #835 is merged

First of all, good work on this fix @carterworks , anything Identity & cookies can be painful.
Question:
It seems that both of these exceptions can/are being handled by Edge:

Non-matching ORG Id
Using third party edge domain

Should we test to see if Konductor already returns an error for mismatched Org ID & Datastream ID, and how it handles 2, and simply log their error?
I'm asking because we are working towards a future where Alloy won't even care about the Identity cookie, so why not try to simplify this logic now.
Thoughts?
@jonsnyder @ninaceban

@jfkhoury
From what I can tell, Konductor has an error message for missing/incorrect datastream IDs, but not for organization IDs.
Also, I cannot figure out a way to block cookies from being set. I've tried iframe stuff (can't figure out how to run TestCafé in both a browser page and an iframe inside that page at the same time), content security policy stuff (can't block cookies with CSP–you can only block third-party cookies with SameSite=None attributes on the cookies and that is independent from CSP) and regular third party domain usage (Konductor just sends a state:store payload when the apex doesn't match the edge, so what would have been a third-party cookie when set by the Set-Cookie response header becomes a first-party cookie when set via Javascript). I'm leaning towards giving up since I don't think the small benefit of a test for that fail case is worth spending additional days on it.

Closed in favor of the more simple PR #846, which adds a functional test and tweaks the language of the error message slightly.


Clicking the centre of the trash can icon does not work; only the left/right edge.

Bug Report
Please fill out this information:

Browser version: 70.0
Extension version: 1.2.7
Operating system: Windows 10.

What is happening?
In the list view where all domain-specific notes are listed, there are trash can icons, but clicking at the centre of the icon did not work. I tried click at the right edge of the icon, and then it worked. Maybe because of the page zoom or high-DPI?
What are the steps to reproduce the bug?
Click the centre of a trash can icon.
Note
Additionally, the font of the list and the icon size are too small. I have increased the font size in the settings, but that seems to be applied only to the note-taking area, not the list.

This is a duplicate of #17
Essentially, the click event is triggering on the icon itself, instead of the container the icon is in.


re-work access handling

Various cleanups and fixes to handling of region accesses, including:

there should be no reason for us to split accesses into 1/2/4/8 byte accesses:
in general, the client will have already be doing that, and if not, there's no
particular reason we should be the ones to split up such larger accesses.

use a callback for PCI config space reads and writes if one is provided (needs
more work for capabilities)

Signed-off-by: John Levon<EMAIL_ADDRESS>

I'm still not convinced we need the 1/2/4/8 splitting code (removed by the libvfio-user.c changes). The client should be automatically doing this for any typical access, and other cases (such as non-PCI devices) it seems weird to enforce.
Can we revisit this after this patchset (I have a bunch more access changes that involve splitting)

I'm still not convinced we need the 1/2/4/8 splitting code (removed by the libvfio-user.c changes). The client should be automatically doing this for any typical access, and other cases (such as non-PCI devices) it seems weird to enforce.
Can we revisit this after this patchset (I have a bunch more access changes that involve splitting)

I'm still not convinced we need the 1/2/4/8 splitting code (removed by the libvfio-user.c changes). The client should be automatically doing this for any typical access, and other cases (such as non-PCI devices) it seems weird to enforce.
Can we revisit this after this patchset (I have a bunch more access changes that involve splitting)

Sure that's faily trivial to re-add.

I'm still not convinced we need the 1/2/4/8 splitting code (removed by the libvfio-user.c changes). The client should be automatically doing this for any typical access, and other cases (such as non-PCI devices) it seems weird to enforce.
Can we revisit this after this patchset (I have a bunch more access changes that involve splitting)

Sure that's faily trivial to re-add.

I went a little bit further with the region setup mmap thing - I removed the MMAP flag - it didn't seem to have any value, since the 'fd' parameter defines whether the region is mappable. Also fixed up region info such that we weren't silently relying on matching the VFIO region flag values.
PS I'll wait for Swapnil to take a look

I went a little bit further with the region setup mmap thing - I removed the MMAP flag - it didn't seem to have any value, since the 'fd' parameter defines whether the region is mappable. Also fixed up region info such that we weren't silently relying on matching the VFIO region flag values.
PS I'll wait for Swapnil to take a look

@tmakatos sorry, noticed another little fix

@tmakatos sorry, noticed another little fix


feat(@angular-devkit/build-angular): enhance Sass rebasing importer for resources URL defined in variables and handling of external paths

This commit introduces enhancements to the Sass rebasing importer, enabling it to resolve resources whose paths are stored in variables or namespaced variables. Also this addresses an issue where URL paths in Sass and SCSS files, flagged as external, were incorrectly rebased, leading to resolution errors.
Closes #27445 and closes #27007

I think if you update the resolve call in the first commit (1576335#diff-9c5b4495153e90ce431014a888e42237c40e7be76a89ed2866a012615f578f39R61-R67) to adjust the resolveDir instead of the input path you should be able to remove the need for the second fix commit.


Interruption doesn't work on fibers which have used bracket

It's not that easy to reproduce but here's the minimal example I arrived at:
object X extends App with RTS {
  import scala.concurrent.duration._

  val c = new AtomicInteger(0)

  val test: IO[Throwable, Unit] = IO
    .syncThrowable {
      println(System.currentTimeMillis() + " working")
      Thread.sleep(100)
      if (c.incrementAndGet() <= 1) {
        throw new RuntimeException("x")
      } else {
        ()
      }
    }
    .forever
    .ensuring(IO.sync(println("CLEANUP; RESTARTING")))
    .attempt
    .forever

  unsafePerformIO[Void, Unit](
    test
      .fork[Void]
      .flatMap(f => IO.sleep[Void](1.second) *> f.interrupt(new RuntimeException("y"))))
}

Even though the fiber is interrupted after 1 second, it ends up printing working infinitely. Interruption works correctly if the bracket (ensuring) never runs (so if throw new RuntimeException("x") is never executed) or if there's no bracket at all.
The double-forever-loop is there as the original concept was to acquire a resource, use it in a loop, and if there's an error clean up and acquire again (hence the second loop).

@wi101 We'll have to clean this up and put it into a test.
As far as I can tell from a preliminary look, the correct behavior should be:

Prints out "working"
Prints out "CLEANUP; RESTARTING"
Prints out "working" about 9 times (it's non-deterministic exactly how many)
Interrupts and terminates with RuntimeException("y") after printing "CLEANUP; RESTARTING" one more time

We can eliminate the print lines, reduce the timeouts, and see if we can chop anything else out in the test case.

This test case without throw new RuntimeException("x") works correctly.
We have a problem if the release had a failed result in Bracket.
@jdegoes I will commit the change, the behavior now is exactly like you mentioned ! (in the third step "working" appears exactly 9 times! 😁 )

@jdegoes I couldn't apply the change because the test cases in RTSSpec of brackets will fail,
I wanted to do this in RTS at the case of Bracket
             curIo = for {
                      a <- (for {
                            a <- io.acquire
                            _ <- IO.sync(ref.set(a))
                          } yield a).uninterruptibly
                      b <- io.use(a).attempt
                      _ <- ((b match {
                            case Right(r) => io.release(ExitResult.Completed(r), a)[E]
                            case Left(e)  => io.release(ExitResult.Failed(e), a)[E]
                          }) <* IO.sync(ref.set(null))).uninterruptibly
                    } yield b

@wi101 Maybe we will defer this one a bit until after the Finalizer changes are in. I have a feeling I may know what the core bug is here (but maybe not), which is non-interruption.

@adamw Should be fixed now. Was introduced in the ZIO port but now there's a regression test case in there.

@jdegoes thanks! this passes, though the 'big' test (https://github.com/softwaremill/akka-vs-scalaz/blob/master/core/src/test/scala/com/softwaremill/supervise/ZioSuperviseTest.scala) still fails (the interruption doesn't stop the fiber). I'll try to create another test case next week.